-
Notifications
You must be signed in to change notification settings - Fork 75
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
Feature/add motorsport magazin #456
Feature/add motorsport magazin #456
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for adding the motorsport magazin. Be sure to run black .
before your final commit, otherwise the tests won't go through :)
Co-authored-by: Adrian Breiding <ad123br@gmail.com>
Hi! @addie9800 |
Thanks a lot ;) this looks really good. Black seems really weird, when I run black locally it reformats the file back. But I'll just approve the workflow and we'll see if the test go through. |
Sounds good! |
@lsch0lz Could you tell me what version of black is installed in your environment? @addie9800 The tests won't trigger. The last commit on this branch 29c4a5c was auto-committed and due to GitHubs limitations, those won't trigger workflow runs. |
@MaxDall I have the newest black version (24.4.2) installed. Just checked and |
@lsch0lz Thanks :) We pinned the |
# Conflicts: # src/fundus/publishers/de/__init__.py
Hi!
I added the MotorSport Magazin LINK to fundus 🚀
Note: Black reformatted several files, that I didn't intent to touch. Let me know, if thats OK for you