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

Only sign and publish once at the join point #717

Merged
merged 4 commits into from
Mar 21, 2024
Merged

Conversation

mmitche
Copy link
Member

@mmitche mmitche commented Mar 20, 2024

All the artifacts are present, so we only need to sign once

@steveisok steveisok self-requested a review March 20, 2024 22:10
eng/azure-pipelines.yml Outdated Show resolved Hide resolved
@mmitche
Copy link
Member Author

mmitche commented Mar 20, 2024

@akoeplinger
Copy link
Member

@mmitche that's a dotnet/runtime build and the failure is dotnet/runtime#53420

@mmitche
Copy link
Member Author

mmitche commented Mar 20, 2024

@mmitche that's a dotnet/runtime build and the failure is dotnet/runtime#53420

doh https://dev.azure.com/dnceng/internal/_build/results?buildId=2409997&view=results

@lewing lewing merged commit 9a65e4b into dotnet:main Mar 21, 2024
13 checks passed
dkurepa pushed a commit to dkurepa/emsdk that referenced this pull request Mar 21, 2024
* Only run publish at join point (dotnet#714)

(cherry picked from commit 1ed00a9)

* Only sign and publish once

* Set asset manifest file name
mmitche added a commit that referenced this pull request Mar 21, 2024
* Only run publish at join point (#714)

(cherry picked from commit 1ed00a9)

* Only sign and publish once

* Set asset manifest file name

Co-authored-by: Matt Mitchell <mmitche@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants