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

Placed default_suggestion_message on its own line. #1326

Merged
merged 2 commits into from
Sep 26, 2024

Conversation

kmvanbrunt
Copy link
Member

Reformatted how default_suggestion_message appears.

Before

hist is not a recognized command, alias, or macro Did you mean history?

After

hist is not a recognized command, alias, or macro.
Did you mean history?

@tleonhardt
Copy link
Member

Looks like a lot of tests are failing

@kmvanbrunt
Copy link
Member Author

@tleonhardt

Looks like a lot of tests are failing

The macOS code coverage uploads have been failing for over a week now. I don't know how to fix them. All the tests pass but the coverage results never get uploaded.

This is the error we're receiving.

{'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 1076s.', code='throttled')}
Error: 429 Client Error: Too Many Requests for url: https://codecov.io/upload/v2?commit=8e5...

@tleonhardt tleonhardt merged commit 3d7a1e7 into master Sep 26, 2024
40 checks passed
@kmvanbrunt kmvanbrunt deleted the reformat_suggested_error branch September 26, 2024 03:00
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