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

Minor Maintenance Tasks #20

Merged
merged 2 commits into from
Jul 8, 2024
Merged

Minor Maintenance Tasks #20

merged 2 commits into from
Jul 8, 2024

Conversation

JacksonBurns
Copy link
Owner

No description provided.

@JacksonBurns JacksonBurns force-pushed the patch/ci_network_failures branch from 5b48bbd to 25fd50e Compare July 8, 2024 15:19
@JacksonBurns JacksonBurns force-pushed the patch/ci_network_failures branch from a4c1295 to 131a94e Compare July 8, 2024 15:28
@JacksonBurns JacksonBurns requested a review from jonwzheng July 8, 2024 15:28
@JacksonBurns
Copy link
Owner Author

@jonwzheng please review! Just made a couple small commits to remove an unused test attribute and added automatic retries (since network failures cause the CI to fail sometimes, and I don't want to keep manually retrying them).

Copy link
Collaborator

@jonwzheng jonwzheng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@JacksonBurns JacksonBurns merged commit d85468e into main Jul 8, 2024
18 checks passed
@JacksonBurns JacksonBurns deleted the patch/ci_network_failures branch July 8, 2024 15:35
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.

2 participants