-
Notifications
You must be signed in to change notification settings - Fork 9
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
Annotate warnings #68
base: main
Are you sure you want to change the base?
Annotate warnings #68
Conversation
20f27ab
to
a70c269
Compare
87145ee
to
ae9b957
Compare
Yeah, I think |
Done 494dca7! |
this is ready for another review @henryiii 🙂 |
6aaf4d7
to
5d4405e
Compare
5d4405e
to
74a4dfe
Compare
- Refactor workflow command generation - Skip warning annotations in Pytest < 6.0.0
74a4dfe
to
dd855a1
Compare
FYI, we need to hide our warnings to get though CI. Mostly on pytest 6. |
with contextlib.suppress(ValueError): | ||
filesystempath = os.path.relpath(filesystempath) |
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.
@henryiii this lets Windows tests pass now
--exclude-warning-annotations
option to exclude warning annotationsCloses #45