Skip to content

Combine catalogd and operator-controller Makefiles #1340

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
Closed
Tracked by #1646
OchiengEd opened this issue Oct 3, 2024 · 3 comments
Closed
Tracked by #1646

Combine catalogd and operator-controller Makefiles #1340

OchiengEd opened this issue Oct 3, 2024 · 3 comments
Assignees
@OchiengEd

Comments

@OchiengEd
Copy link
Contributor

Combine the Makefiles in the root operator-controller repository and the one in the catalogd directory.
@OchiengEd OchiengEd mentioned this issue Oct 3, 2024
Closed
11 tasks
@jsm84
Copy link
Contributor

jsm84 commented Nov 21, 2024 •
edited
Loading

Linked to internal Jira story EET-4418.

@camilamacedo86 camilamacedo86 mentioned this issue Jan 22, 2025
Closed
8 tasks
@bentito
Copy link
Contributor

bentito commented Jan 23, 2025

Couple things I thought of when looking at this story:

  • Don't forget to update docs for renamed targets
  • Don't forget to update Github actions for renamed targets
  • Don't forget to update hack/helper scripts for renamed targets

camilamacedo86 added a commit to camilamacedo86/operator-controller that referenced this issue Feb 10, 2025
@camilamacedo86
[Monorepo]: Move build-container and kind-load targets to root Makefile
a3ba500 
Partial cleanup/combine the Makefile: operator-framework#1340
camilamacedo86 added a commit to camilamacedo86/operator-controller that referenced this issue Feb 10, 2025
@camilamacedo86
[Monorepo]: Move build-container and kind-load targets to root Makefi…
2a75952 
…le and remove unused values from catalogd/Makefile

Partial cleanup/combine the Makefile: operator-framework#1340
@camilamacedo86 camilamacedo86 mentioned this issue Feb 10, 2025
Closed
4 tasks
@OchiengEd OchiengEd mentioned this issue Feb 24, 2025
Merged
4 tasks
@camilamacedo86
Copy link
Contributor

It is done I am closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OchiengEd OchiengEd

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@OchiengEd @LalatenduMohanty @bentito @camilamacedo86 @jsm84