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.

Sign up for GitHub

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

Jump to bottom

Fix potential multiple warnings in CI #77

Merged
merged 1 commit into from
Feb 2, 2022
Merged

Fix potential multiple warnings in CI #77

merged 1 commit into from
Feb 2, 2022

Conversation

oliversalzburg
Copy link
Contributor

I would hope the fix is as simple as this :)

Fixes #75

@timvink
Copy link
Owner

timvink commented Feb 2, 2022

Thanks, much appreciated!

@timvink timvink merged commit bfb9773 into timvink:master Feb 2, 2022
@timvink
Copy link
Owner

timvink commented Feb 2, 2022

I've also added your suggestion to change GIT_DEPTH to 0 for gitlab in 873a241

@timvink
Copy link
Owner

timvink commented Feb 2, 2022

FYI, I'll release it to pypi when I have some other issues completed also

@oliversalzburg
Copy link
Contributor Author

Awesome. Thanks :)

@oliversalzburg oliversalzburg deleted the fix/ci-warnings branch February 2, 2022 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Multiple/wrong warnings printed in CI
2 participants
@oliversalzburg @timvink