-
Notifications
You must be signed in to change notification settings - Fork 7.8k
Accept string values for allow_experimental_analyzer/enable_analyzer #86109
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
Conversation
|
Workflow [PR], commit [3351274] Summary: ❌
|
9583a58 to
5f8dfe1
Compare
@maxknv what was the precondition? can you PTAL? |
All settings can be passed as strings except for this one, due to manual check in the code, which leads to "Bad get: has String, requested UInt64." before this patch.
5f8dfe1 to
3351274
Compare
That's normal. Multiple jobs are updating the overall CI report concurrently. |
|
But it fails CI, let's add retries at least? |
|
CI issues are known |
469f638
This is the reason for the job error: I have an issue to fix the timeout handling to make this error visible #85444 |
|
@maxknv thank you. Let's maybe increase timeout for LTO builds temporary (to i.e. 3h) ? |
All settings can be passed as strings except for this one, due to manual check in the code, which leads to "Bad get: has String, requested UInt64." before this patch.
Changelog category (leave one):
Fixes: azat/chdig#146