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

list.json AnkeReichert #10

Merged
merged 5 commits into from Aug 2, 2023
Merged

Conversation

mrStronge
Copy link
Contributor

No description provided.

@ahaeslich
Copy link
Owner

Beste Commit-Nachricht bisher 💖.

Btw: Mich würde tatsächlich interessieren, wie viele Einträge auf der Liste von dir stammen 😁

@ahaeslich
Copy link
Owner

Gerade auch erst den Branch-Namen entdeckt ... love it 😁💖

@christophervoigt
Copy link
Collaborator

@ahaeslich Ich muss mir auf jeden Fall nochmal den Test anschauen... der schlägt eigentlich fehlt, wirft aber keinen Error in der Pipeline: https://github.com/ahaeslich/ankelegt/actions/runs/5579805464/jobs/10209093302?pr=10#step:5:14

@mrStronge
Copy link
Contributor Author

Weil die Wortspiele spitze sind. Gitlab hat eben Sinn für Humor. BTW stand in den Regeln mMn. nicht, dass das Wort mit Anke beginnen muss.

@christophervoigt
Copy link
Collaborator

christophervoigt commented Jul 18, 2023

BTW stand in den Regeln mMn. nicht, dass das Wort mit Anke beginnen muss.

Ja, das stimmt. In den Regeln steht, dass das Wort "Anke" enthalten muss. Da du aber der erste mit einem neuen Wortbeginn bist, müsstest du dann entsprechend auch die Regex für den Test anpassen 😉

@ahaeslich
Copy link
Owner

@ahaeslich Ich muss mir auf jeden Fall nochmal den Test anschauen... der schlägt eigentlich fehlt, wirft aber keinen Error in der Pipeline: https://github.com/ahaeslich/ankelegt/actions/runs/5579805464/jobs/10209093302?pr=10#step:5:14

@chlorophyllkid: Done ✅

@ahaeslich
Copy link
Owner

@mrStronge magst du den RegEx noch anpassen?

@mrStronge
Copy link
Contributor Author

@ahaeslich done

@ahaeslich ahaeslich merged commit 8e4b6d1 into ahaeslich:main Aug 2, 2023
3 checks passed
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.

None yet

3 participants