WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@acornett21
Copy link
Contributor

@acornett21 acornett21 commented Nov 17, 2025

Merge Request Checklists

  • Development is done in feature branches
  • Code changes are submitted as pull request into a primary branch [Provide reason for non-primary branch submissions]
  • Code changes are covered with unit and integration tests.
  • Code passes all automated code tests:
    • Linting
    • Code formatter - Black
    • Security scanners
    • Unit tests
    • Integration tests
  • Code is reviewed by at least 1 team member
  • Pull request is tagged with "risk/good-to-go" label for minor changes
❯ crane digest quay.io/opdev/preflight:1.14.1
sha256:57f74e3d1ad72bfd11cf888eae99f8cad4d75c25515413a7c30ad9963aed2741

❯ crane digest quay.io/opdev/preflight:1.15.2
sha256:6e857a015829a53e2139b3f857816c91ff9bf0b37a44d4e473ad92f8b878c1c9

@acornett21 acornett21 added risk/good-to-go Indicates a small risk change ready-for-testing labels Nov 17, 2025
Copy link
Contributor

@komish komish left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 18, 2025
Copy link
Contributor

@bcrochet bcrochet left a comment

Choose a reason for hiding this comment

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

/lgtm

@acornett21 acornett21 added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 19, 2025
@acornett21 acornett21 changed the title update preflight sha for 1.15.1 release update preflight sha for 1.15.2 release Nov 20, 2025
@acornett21 acornett21 removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 20, 2025
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 20, 2025
Copy link
Contributor

@komish komish left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 20, 2025
@acornett21 acornett21 merged commit c861792 into main Nov 20, 2025
8 of 12 checks passed
@acornett21 acornett21 deleted the update_preflight_sha branch November 20, 2025 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. ready-for-testing risk/good-to-go Indicates a small risk change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants