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

feat: add finch version output to support-bundle #1124

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

swagatbora90
Copy link
Contributor

Dumps finch version output to a file when generating finch support-bundle

Testing done:

make test-unit and make test-e2e

  • [x ] I've reviewed the guidance in CONTRIBUTING.md

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@swagatbora90 swagatbora90 force-pushed the add-version-to-support branch from cc02ef3 to a2c089d Compare October 3, 2024 20:52
@swagatbora90 swagatbora90 changed the title Add finch version output to support-bundle feat: add finch version output to support-bundle Oct 3, 2024
Copy link
Member

@pendo324 pendo324 left a comment

Choose a reason for hiding this comment

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

One question, otherwise LGTM pending the linter fixes

pkg/support/support.go Outdated Show resolved Hide resolved
@swagatbora90 swagatbora90 force-pushed the add-version-to-support branch 2 times, most recently from 0d23015 to a16be78 Compare October 3, 2024 22:48
@swagatbora90 swagatbora90 force-pushed the add-version-to-support branch from a16be78 to 77fa357 Compare October 4, 2024 18:20
@swagatbora90 swagatbora90 merged commit 33d519b into runfinch:main Oct 7, 2024
28 checks 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.

3 participants