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

Code Clean Up for Platfrom Controller #1377

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

SuyogShinde942
Copy link
Contributor

This PR resolves issues #1371 and #1370:

Cleanup Unused Resource Policy Controller (#1371):

Removed the unused resourcepolicycontroller.go file.
Deleted commented-out references to resourcepolicycontroller in main.go, as it was never utilized.
Enhancements in Resource Composition Handling (#1370):

Improved PlatformController by ensuring the handleCRD method registers the CRD defined in the ResourceComposition spec.

@devdattakulkarni
Copy link
Contributor

@SuyogShinde942 Will it be possible to squash the 6 commits into a single commit?
You can keep the current commit message.

@SuyogShinde942
Copy link
Contributor Author

sure

@SuyogShinde942 SuyogShinde942 force-pushed the master branch 3 times, most recently from cdbd3eb to a68448b Compare November 27, 2024 14:19
Remove commented code for platformcontroller.go

remove commented code

remove commented code

remove unused fucntions GetValuesTypes, getChartValueTypes,flatten

remove unused imports

Remove unused imports, functions, and references; clean up commented code for improved readability.
@SuyogShinde942
Copy link
Contributor Author

@devdattakulkarni Can you please check this

@devdattakulkarni
Copy link
Contributor

@SuyogShinde942 Looks good 👍

@devdattakulkarni devdattakulkarni merged commit aa45ad0 into cloud-ark:master Nov 27, 2024
1 check passed
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