Skip to content

Conversation

@kaizencc
Copy link
Contributor

@kaizencc kaizencc commented Feb 21, 2025

runs are currently failing because runs-on is required: https://github.com/aws/aws-cdk-cli/actions/runs/13461282048


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

});

this.workflow.addJob('collect', {
runsOn: ['ubuntu-latest'],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mrgrain we should make this either required in projen or default to ubuntu-latest right

Copy link
Contributor

@mrgrain mrgrain Feb 21, 2025

Choose a reason for hiding this comment

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

Huh. That might have changed tbh. But yes.

@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@kaizencc kaizencc disabled auto-merge February 21, 2025 23:34
@kaizencc kaizencc enabled auto-merge February 21, 2025 23:34
@kaizencc kaizencc added the pr/exempt-integ-test Skips the integ test steps if set. label Feb 21, 2025
@kaizencc kaizencc added this pull request to the merge queue Feb 21, 2025
Merged via the queue into main with commit 104ca7f Feb 22, 2025
11 checks passed
@kaizencc kaizencc deleted the conroy/codecov branch February 22, 2025 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/exempt-integ-test Skips the integ test steps if set.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants