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. #5099

Closed
wants to merge 2 commits into from

Conversation

pulumi-bot
Copy link
Contributor

This PR was automatically generated by the update-workflows-ecosystem-providers workflow in the pulumi/ci-mgmt repo, from commit 62fdb1f928969ade9cecbec1408c5ae5963edff2.

@pulumi-bot pulumi-bot added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Jan 14, 2025
@pulumi-bot pulumi-bot enabled auto-merge (squash) January 14, 2025 05:09
Copy link

Does the PR have any schema changes?

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

Maintainer note: consult the runbook for dealing with any breaking changes.

Copy link

codecov bot commented Jan 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 24.46%. Comparing base (bf974ad) to head (3c624df).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5099   +/-   ##
=======================================
  Coverage   24.46%   24.46%           
=======================================
  Files         361      361           
  Lines      144037   144037           
=======================================
  Hits        35243    35243           
  Misses     108696   108696           
  Partials       98       98           

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

@t0yv0 t0yv0 self-requested a review January 14, 2025 14:14
@t0yv0
Copy link
Member

t0yv0 commented Jan 14, 2025

.mk/minimal schema is unhappy possibly with my latest ci-mgmt changes, having a look.

bin/darwin-amd64/$(PROVIDER): TARGET := darwin-amd64
bin/darwin-arm64/$(PROVIDER): TARGET := darwin-arm64
bin/windows-amd64/$(PROVIDER).exe: TARGET := windows-amd64
bin/linux-amd64/$(PROVIDER): export GOOS := linux
Copy link
Member

Choose a reason for hiding this comment

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

I think I know what the problem is. The export GOOS bit got exported all the way to override the go generate call in minimal_schema.mk. I can repro locally if I run GOOS=linux go generate e.g. specify the wrong OS.

@t0yv0
Copy link
Member

t0yv0 commented Jan 14, 2025

Needs #5107

@pulumi-bot pulumi-bot closed this Jan 15, 2025
auto-merge was automatically disabled January 15, 2025 05:09

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