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.

Sign up for GitHub

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

Jump to bottom

Fix Build Scan publication #37663

Merged
merged 3 commits into from
Dec 12, 2023
Merged

Fix Build Scan publication #37663

merged 3 commits into from
Dec 12, 2023

Conversation

jprinet
Copy link
Contributor

@jprinet jprinet commented Dec 11, 2023

  • Use env.RUNNER_TEMP to store Build Scan dumps copy
  • Add permissions to delete an artifact to the publication workflow
  • Fix build-metadata.json location

@quarkus-bot quarkus-bot bot added the area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure label Dec 11, 2023
@jprinet jprinet mentioned this pull request Dec 11, 2023
Merged
@gsmet
Copy link
Member

gsmet commented Dec 11, 2023

Thanks, I'll have a proper CI run this time to avoid any bad surprises :).

@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Dec 11, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@gsmet gsmet merged commit 506e0fa into quarkusio:main Dec 12, 2023
49 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.7 - main milestone Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gsmet gsmet gsmet approved these changes

Assignees
No one assigned
Labels
area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure
Projects
None yet
Milestone
3.7.0.CR1
Development

Successfully merging this pull request may close these issues.
2 participants
@jprinet @gsmet