Skip to content

Conversation

@jpobst
Copy link
Contributor

@jpobst jpobst commented Nov 7, 2022

CodeQL was added in #1057, however it does not complete within our current 20 minute timeout. The documentation recommends tripling the timeout, so set it to 60.

@jpobst jpobst marked this pull request as ready for review November 7, 2022 23:44
@jpobst jpobst requested a review from pjcollins November 7, 2022 23:44
Copy link
Member

@pjcollins pjcollins left a comment

Choose a reason for hiding this comment

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

This seems good for now. We can track future runs and consider moving this into a new job if it is dramatically affecting build times.

@jpobst jpobst merged commit 984711a into main Nov 8, 2022
@jpobst jpobst deleted the jpobst-patch-1 branch November 8, 2022 14:48
jonpryor pushed a commit to dotnet/android that referenced this pull request Nov 11, 2022
Changes: dotnet/java-interop@c6c487b...984711a

  * dotnet/java-interop@984711a3: [ci] Increase timeout to 60 mins for CodeQL (dotnet/java-interop#1058)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants