Skip to content

Able to use project components as golang library #718

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

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

dmvolod
Copy link
Contributor

@dmvolod dmvolod commented Jan 9, 2025

This PR moves manifests generate related code from the cmd package to the manifest and make Generate manifests function public.

fixes #717

@dmvolod dmvolod changed the title Able to use this project components as golang library Able to use project components as golang library Jan 9, 2025
@yxxhero
Copy link
Collaborator

yxxhero commented Jan 15, 2025

@mumoshu

@yxxhero
Copy link
Collaborator

yxxhero commented Jan 15, 2025

@databus23

@dmvolod
Copy link
Contributor Author

dmvolod commented Jan 24, 2025

Hi @yxxhero
Is there any chance to get it merged? I'm sure, it will not break key plugin functionality.
Thanks in advance.

@yxxhero
Copy link
Collaborator

yxxhero commented Jan 27, 2025

@mumoshu @databus23 WDYT? thanks so much.

@yxxhero
Copy link
Collaborator

yxxhero commented Feb 3, 2025

LGTM

@yxxhero
Copy link
Collaborator

yxxhero commented Feb 3, 2025

@dmvolod all is good. could you squash your commits and signoff your commits? then I will merge this PR.

Copy link
Collaborator

@yxxhero yxxhero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yxxhero yxxhero self-assigned this Feb 3, 2025
@dmvolod
Copy link
Contributor Author

dmvolod commented Feb 3, 2025

@dmvolod all is good. could you squash your commits and signoff your commits? then I will merge this PR.

Thanks, @yxxhero for taking care PR.
Commits squashed.

@yxxhero
Copy link
Collaborator

yxxhero commented Feb 3, 2025

please signoff your commit

@dmvolod
Copy link
Contributor Author

dmvolod commented Feb 3, 2025

please signoff your commit

Done.

@yxxhero yxxhero merged commit 815a691 into databus23:master Feb 3, 2025
16 checks passed
@abhijith-darshan
Copy link

When will this be part of the next release?

@yxxhero
Copy link
Collaborator

yxxhero commented Feb 4, 2025

@abhijith-darshan today.

@yxxhero
Copy link
Collaborator

yxxhero commented Feb 4, 2025

@yxxhero
Copy link
Collaborator

yxxhero commented Feb 4, 2025

@abhijith-darshan released. and need feedbacks.

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.

Able to use project components as golang library
3 participants