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

Harden automated build #311

Merged
merged 2 commits into from
Jul 11, 2023
Merged

Harden automated build #311

merged 2 commits into from
Jul 11, 2023

Conversation

mfocko
Copy link
Member

@mfocko mfocko commented Jul 10, 2023

  • Expose the tokens only to the steps where the tokens are required.
  • Do not merge the script used for notifications on the PRs.

@mfocko mfocko added kind/bug Something isn't working. high-prio High priority. security Security concern. labels Jul 10, 2023
@usercont-release-bot
Copy link

usercont-release-bot commented Jul 10, 2023 •
edited
Loading

Preview: https://packit-dashboard-pr-311.surge.sh (deployed at Tue 11 Jul 2023, 16:50 UTC)

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/95add5bb419a4bf793dbb9589e14bb36

✔️ pre-commit SUCCESS in 1m 31s

@mfocko mfocko closed this Jul 11, 2023
@mfocko mfocko deleted the fix-merging branch July 11, 2023 11:28
@mfocko mfocko restored the fix-merging branch July 11, 2023 16:47
@mfocko mfocko reopened this Jul 11, 2023
@mfocko
Copy link
Member Author

mfocko commented Jul 11, 2023

I'm a pepege…

@mfocko mfocko enabled auto-merge July 11, 2023 16:49
@mfocko mfocko added this pull request to the merge queue Jul 11, 2023
Merged via the queue into packit:main with commit 983335e Jul 11, 2023
1 check passed
@mfocko mfocko deleted the fix-merging branch July 11, 2023 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jpopelka jpopelka jpopelka approved these changes

@lachmanfrantisek lachmanfrantisek lachmanfrantisek approved these changes

@lbarcziova lbarcziova lbarcziova approved these changes

Assignees
No one assigned
Labels
high-prio High priority. kind/bug Something isn't working. security Security concern.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@mfocko @usercont-release-bot @jpopelka @lachmanfrantisek @lbarcziova