Skip to content

Conversation

@averissimo
Copy link
Contributor

Pull Request

Changes description

Suggestions from talk with @vedhav

@averissimo averissimo force-pushed the jk-vignette@pre-release-cleanup@main branch from b241683 to 6cd34d7 Compare January 12, 2024 08:43
@averissimo averissimo requested a review from vedhav January 12, 2024 08:43
Copy link
Contributor

@vedhav vedhav left a comment

Choose a reason for hiding this comment

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

Thanks LGTM

@averissimo
Copy link
Contributor Author

@kartikeyakirar should this be merged to pre-release branch or should it target main?

@kartikeyakirar
Copy link
Contributor

should this be merged to pre-release branch or should it target main?

its a part of the cleanup process, I believe it would be beneficial to merge it with the pre-release.

Co-authored-by: kartikeya kirar <kirar.kartikeya1@gmail.com>
Signed-off-by: André Veríssimo <211358+averissimo@users.noreply.github.com>
@averissimo averissimo merged commit ea6ddea into pre-release-cleanup@main Jan 12, 2024
@averissimo averissimo deleted the jk-vignette@pre-release-cleanup@main branch January 12, 2024 10:01
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.

4 participants