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

Add flag to show test status as a word next to the icons #22

Merged
3 commits merged into from
Dec 31, 2021

Conversation

MartinPetkov
Copy link
Contributor

Please describe the change you are making

It may not be obvious what the icons represent, particularly the one for "SKIP". This adds a bit of documentation and a flag to show the status as a word next to the icon.

Your code will be released under the Unlicense into the public domain for everyone to use for any purpose. Are you in the position, and are you willing to release your code under this license?

Yes.

It may not be obvious what the icons represent, particularly the one for "SKIP". This adds a bit of documentation and a flag to show the status as a word next to the icon.
@MartinPetkov MartinPetkov changed the title Status words Add flag to show test status as a word next to the icons Dec 29, 2021
@MartinPetkov
Copy link
Contributor Author

Not sure if I'm able to assign reviewers but @janosdebugs.

@MartinPetkov
Copy link
Contributor Author

PS thank you for making this project, I like the nicer output and am using it in my dev setup now :)

@ghost ghost self-requested a review December 30, 2021 12:19
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Hey @MartinPetkov thank you very much for this, this is awesome! I would suggest a few minor changes.

.gotestfmt/github/package.gotpl Outdated Show resolved Hide resolved
.gotestfmt/package.gotpl Outdated Show resolved Hide resolved
.gotestfmt/teamcity/package.gotpl Outdated Show resolved Hide resolved
@ghost ghost added the enhancement New feature or request label Dec 30, 2021
@ghost ghost enabled auto-merge (squash) December 30, 2021 19:59
auto-merge was automatically disabled December 31, 2021 03:53

Head branch was pushed to by a user without write access

@MartinPetkov MartinPetkov requested a review from a user December 31, 2021 03:55
@ghost ghost enabled auto-merge (squash) December 31, 2021 07:38
@ghost ghost merged commit f5f1fee into GoTestTools:main Dec 31, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

1 participant