Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the MIGs and AHB XMLs as PRIVATE submodule #69

Closed
7 tasks done
hf-kklein opened this issue Jan 9, 2025 · 2 comments · Fixed by #70
Closed
7 tasks done

Add the MIGs and AHB XMLs as PRIVATE submodule #69

hf-kklein opened this issue Jan 9, 2025 · 2 comments · Fixed by #70
Assignees

Comments

@hf-kklein
Copy link
Contributor

hf-kklein commented Jan 9, 2025

  • submodule auf https://github.com/Hochfrequenz/xml-migs-and-ahbs hinzufügen
  • einen PAT (Personal Access Token) anlegen, der zugriff auf das repo hat => https://github.com/Hochfrequenz/xml-fundamend-python/settings/secrets/actions/XML_SUBMODULE_PAT
  • die CI so umbauen, dass das submodule mit ausgechekt wird und den PAT dazu verwendet (orientier dich zum beispiel am companystylesheet repo, siehe nächster kommentar)
  • einen PR öffnen
  • einen Test schreiben, der alle XML AHbs testweise einliest. Der test darf keine daten aus den XMLs hierher (in das öffenltich repo leaken, oder wir machen es private, wenn's nicht anders geht)
  • einen Test schreiben, der alle XML MIGs testweise einliest. Der test darf keine daten aus den XMLs hierher (in das öffenltich repo leaken, oder wir machen es private, wenn's nicht anders geht)
  • für alles, was out of the box nicht geht kleinteilige, granulare PRs öffnen, die das jeweilige Problem adressieren
@hf-kklein
Copy link
Contributor Author

hf-kklein commented Jan 9, 2025

@FreddyFox892 FreddyFox892 linked a pull request Jan 13, 2025 that will close this issue
@hf-kklein
Copy link
Contributor Author

fürs protokoll: in #70 gab es nichts, was out of the box gecrashed wäre.

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 a pull request may close this issue.

2 participants