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

Add check for web_catalog_only to Submission #344

Merged

Conversation

mgoerens
Copy link
Contributor

This commits ports the "check_web_catalog_only" from the checkpr package over to the Submission data structure.

@mgoerens mgoerens marked this pull request as draft April 16, 2024 14:48
This was referenced Apr 16, 2024
@mgoerens mgoerens force-pushed the add_owners_check_to_submission branch 2 times, most recently from 31af8db to d67c958 Compare April 19, 2024 13:58
@mgoerens mgoerens force-pushed the add_owners_check_to_submission branch 3 times, most recently from 79d1f38 to 98a4e21 Compare May 30, 2024 20:27
This commits ports the "check_web_catalog_only" from the checkpr package
over to the Submission data structure.

Signed-off-by: Matthias Goerens <mgoerens@redhat.com>
@mgoerens mgoerens force-pushed the add_owners_check_to_submission branch from 98a4e21 to 9e3d4db Compare May 30, 2024 20:37
@mgoerens mgoerens marked this pull request as ready for review May 30, 2024 20:38
@mgoerens mgoerens requested a review from komish May 30, 2024 20:38
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

Thanks @mgoerens

@mgoerens mgoerens merged commit ab127be into openshift-helm-charts:main Jun 3, 2024
9 checks passed
@mgoerens mgoerens deleted the add_owners_check_to_submission branch June 3, 2024 08:49
jsm84 added a commit to jsm84/openshift-helm-charts-dev that referenced this pull request Jun 3, 2024
Summary:
* Standardize action runner versions across workflows by @komish in openshift-helm-charts#339
* Correctly determine the OWNERS file change type by @komish in openshift-helm-charts#340
* Initialize the Submission data structure by @mgoerens in openshift-helm-charts#332
* Allow token access to redhat OWNERS file metadata check automation by @komish in openshift-helm-charts#338
* Remove unused function in 'signed_chart.py' by @mgoerens in openshift-helm-charts#322
* Ping maintainers if the release job fails by @mgoerens in openshift-helm-charts#343
* Rework owners_file.py and add custom exception by @mgoerens in openshift-helm-charts#321
* Add debug information when tests fail to create PR by @mgoerens in openshift-helm-charts#346
* Fix empty PR number in tests logs by @mgoerens in openshift-helm-charts#348
* New chart fixture with plus sign in version by @jsm84 in openshift-helm-charts#349
* Rename crysostat fixture & rm whitespace in provider name by @jsm84 in openshift-helm-charts#351
* Added new test scenario plus-in-version to HC-17 by @jsm84 in openshift-helm-charts#350
* Cleanup unused report fixtures for HC-16 by @jsm84 in charts/development/pull/351
* Added new test scenario plus-in-version to HC-17 by @jsm84 in openshift-helm-charts#350
* Cleanup unused report fixtures for HC-16 by @jsm84 in openshift-helm-charts#352
* Inform caller if web_catatog_only key is missing by @mgoerens in openshift-helm-charts#345
* Add check for web_catalog_only to Submission by @mgoerens in openshift-helm-charts#344
openshift-helm-charts#352
* Inform caller if web_catatog_only key is missing by @mgoerens in openshift-helm-charts#345
* Add check for web_catalog_only to Submission by @mgoerens in openshift-helm-charts#344

**Full Changelog**: openshift-helm-charts/development@1.7.0...1.7.1

Signed-off-by: Josh Manning <19478595+jsm84@users.noreply.github.com>
@jsm84 jsm84 mentioned this pull request Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants