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

fix(newman-reporter-allure): Update README.md - change flag export to reportDir (fixes #992) #1057

Merged
merged 1 commit into from
Jul 15, 2024

Conversation

cryptton2004
Copy link
Contributor

@cryptton2004 cryptton2004 commented Jul 11, 2024

Context

#992

The better solution would've been to allow both for backwards compatibility
Old: --reporter-allure-export
New: --reporter-allure-reportDir

Checklist

  • Sign Allure CLA

Sorry, something went wrong.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@CLAassistant
Copy link

CLAassistant commented Jul 11, 2024

CLA assistant check
All committers have signed the CLA.

@baev baev added the type:docs label Jul 11, 2024
@cryptton2004 cryptton2004 changed the title Update README.md - change flag export to reportDir (fixes #992) fix(newman-reporter-allure): Update README.md - change flag export to reportDir (fixes #992) Jul 12, 2024
@cryptton2004
Copy link
Contributor Author

seems stuck

@baev baev merged commit b17a493 into allure-framework:main Jul 15, 2024
8 of 9 checks passed
@baev
Copy link
Member

baev commented Jul 15, 2024

@cryptton2004 thanks!

@cryptton2004 cryptton2004 deleted the patch-1 branch July 15, 2024 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants