Skip to content

Conversation

@OchiengEd
Copy link
Contributor

Description

Reviewer Checklist

  • API Go Documentation
  • Tests: Unit Tests (and E2E Tests, if appropriate)
  • Comprehensive Commit Messages
  • Links to related GitHub Issue(s)

This PR is related to issue #1341 and attempts to combine the config directories of operator-controller and catalogd into a single directory.

The PR aims to re-use any common files where they may exist but still maintain unique output files at the end of the effort.

Pending

  • Further testing is still required to ensure affected make targets are working as expected following the changes

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 4, 2025
@netlify
Copy link

netlify bot commented Feb 4, 2025

Deploy Preview for olmv1 ready!

Name Link
🔨 Latest commit b69b303
🔍 Latest deploy log https://app.netlify.com/sites/olmv1/deploys/67a26da949f55d0008432859
😎 Deploy Preview https://deploy-preview-1701--olmv1.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@OchiengEd
Copy link
Contributor Author

Closing in favor of smaller PR's

@OchiengEd OchiengEd closed this Feb 6, 2025
@OchiengEd OchiengEd deleted the merge_kustomize branch February 6, 2025 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant