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

Adds an option to collect output from cli runs #212

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

tpyle
Copy link
Contributor

@tpyle tpyle commented Sep 25, 2023

This adds a --output or a -o option to dump the results of the CLI runs into a file.

This also modifies the way requests are run to reduce duplication of logic.

@tpyle tpyle marked this pull request as ready for review September 25, 2023 21:49
@tpyle
Copy link
Contributor Author

tpyle commented Sep 25, 2023

Hello @helloanoop, I've made this PR to fill a need to collect results from cli runs. If you'd like me to create an issue to associate this, I'm happy to do so. Thanks!

@tpyle
Copy link
Contributor Author

tpyle commented Sep 25, 2023

Also, my editor messed with the readme to make it more conformant with markdown syntax, I can revert that if you'd like.

@helloanoop helloanoop merged commit c7aecbe into usebruno:main Sep 26, 2023
1 check passed
@helloanoop
Copy link
Contributor

Hey @tpyle !

This is cool!
I feel this will help devs to write custom scripts that can use the results file and do a variety of things like

  • send mail with the test summary
  • upload data to grafana / any monitoring utility

PR Merged !!

cc @mirkogolze just fyi

helloanoop added a commit that referenced this pull request Sep 26, 2023
@helloanoop
Copy link
Contributor

Bru CLI v0.9.0 has been published to npm 🎉

helloanoop added a commit to usebruno/bruno-docs that referenced this pull request Sep 26, 2023
@tpyle tpyle deleted the feature/output-collection branch September 26, 2023 23:41
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