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.

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

Internal restructure #161

Merged
merged 14 commits into from
Aug 27, 2022
Merged

Internal restructure #161

merged 14 commits into from
Aug 27, 2022

Conversation

ccremer
Copy link
Owner

@ccremer ccremer commented Aug 21, 2022 •
edited

Summary

  • Decouples the commands from any specific CLI library
  • Splits controllers into their own packages
  • Use pipeline library to help better understand the business code flow.

Checklist

  • Categorize the PR by setting a good title and adding one of the labels:
    kind:bug, kind:enhancement, kind:documentation, kind:change, kind:breaking, kind:dependency
    as they show up in the changelog
  • PR contains the label area:operator
  • Link this PR to related issues
  • I have not made any changes in the charts/ directory.

@ccremer ccremer added the kind:change generic change that is neither a bugfix nor a feature label Aug 21, 2022
@ccremer ccremer changed the title Restructure Internal restructure Aug 21, 2022
@ccremer ccremer force-pushed the restructure branch 2 times, most recently from 69d4879 to 9708a89 Compare August 27, 2022 18:50
@ccremer ccremer merged commit 2be94c1 into master Aug 27, 2022
@ccremer ccremer deleted the restructure branch August 27, 2022 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area:operator kind:change generic change that is neither a bugfix nor a feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@ccremer