Skip to content

Port lawdown.py to Python3#12

Merged
ulfgebhardt merged 1 commit intobundestag:masterfrom
jbruechert:lawdown-py3
Mar 27, 2021
Merged

Port lawdown.py to Python3#12
ulfgebhardt merged 1 commit intobundestag:masterfrom
jbruechert:lawdown-py3

Conversation

@jbruechert
Copy link
Copy Markdown
Contributor

No description provided.

@ulfgebhardt
Copy link
Copy Markdown
Member

ulfgebhardt commented Mar 26, 2021

Hey @jbbgameich
ich hab leider keine Ahnung von dem Tool und kann daher das ganze nicht guten Gewissens direkt mergen.
Hättest du Interesse einen Github Workflow für das tool zu bauen, der zumindest mal sicherstellt, dass das ding kompiliert?

Ref:

@jbruechert
Copy link
Copy Markdown
Contributor Author

Da es Python ist gibt es ja leider keine kompilierung. So weit ich es beurteilen kann funktioniert es aber, es fallen ordentliche Markdown Dateien raus.

Ich hatte das Skript jetzt einfach über den output von lawde.py laufen lassen. Reicht dir das zum testen?

Ein GitLab workflow macht glaub ich erst richtig Sinn wenn es unit tests gäbe, aber welche extra für diesen Pull Request zu schreiben wäre wahrcheinlich etwas übertrieben.

@ulfgebhardt
Copy link
Copy Markdown
Member

Also ideal wäre wenn man irgendwie ein "binary" erstellen bzw den interpreter drüberlaufen lassen könnte. Ich habe leider noch nie etwas ernsthaftes mit python gemacht, da heißt es glaube ich packages erstellen oder so.

Ich hab überhaupt nix dagegen das zu mergen. Das Repo ist aktuell ziemlich tot
image

ich bekomme allerdings mit der anstehenden Bundestagswahl mehr Anfragen auf https://github.com/bundestag/gesetze und soweit ich das verstanden habe ist dieses Repo zum generieren des Inhalts zuständig.

Siehe: bundestag/gesetze#59 und bundestag/gesetze#55
Heute habe ich eine Anfrage von @Muehe bekommen bezüglich des Repos.

Ich fände es cool wenn wir es gemeinschaftlich hinbekommen ein script für die github workflows zu schreiben um ähnlich zu den Repos die wir für das democracy Projekt crawlen regelmäßig updates bekommen.

Die Leute von https://github.com/bundestag/offenegesetze.de haben sich da leider noch nicht eingeklinkt um diese Aufgabe zu übernehmen.

Also @jbbgameich : Hast du bock sowas zu machen? Soll ich diesen PR nun mergen?

@jbruechert
Copy link
Copy Markdown
Contributor Author

Ich glaube es sollte einigermaßen sicher sein den PR zu mergen. Das Regelmäßige crawling klingt interessant. Ich habe leider keine Erfahrung mit GitHub actions, aber vielleicht kann ich trotzdem irgendwie helfen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants