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

Update GitHub Actions workflows. #3350

Closed
wants to merge 5 commits into from

Conversation

pulumi-bot
Copy link
Contributor

This PR was automatically generated by the update-native-provider-workflows workflow in the pulumi/ci-mgmt repo, from commit b7f0e100bb0abe6720cb1fd92d053fa66129643b.

@pulumi-bot pulumi-bot added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Dec 5, 2024
@pulumi-bot pulumi-bot enabled auto-merge (squash) December 5, 2024 05:04
Copy link

github-actions bot commented Dec 5, 2024

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

Copy link

codecov bot commented Dec 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.11%. Comparing base (e065015) to head (3360951).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3350   +/-   ##
=======================================
  Coverage   41.11%   41.11%           
=======================================
  Files          85       85           
  Lines       12741    12741           
=======================================
  Hits         5238     5238           
  Misses       7113     7113           
  Partials      390      390           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rquitales
Copy link
Member

Looks like we can't just start checking in the embedded schemas to unblock CI since versions are embedded into schema. This means there'll always be drift/dirty work trees depending on where schema is generated. Seems like we might have to continue not checking in schemas, but have CI touch these embed files so we don't get a compilation error for linting purposes.

@pulumi-bot pulumi-bot closed this Dec 6, 2024
auto-merge was automatically disabled December 6, 2024 05:04

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/no-changelog-required This issue doesn't require a CHANGELOG update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants