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

disable go test verbose flag #4482

Merged
merged 1 commit into from
Aug 4, 2023

Conversation

diego1q2w
Copy link
Contributor

In this pull request, I propose removing the -v flag from our go test command. Currently, our test suite outputs a substantial amount of data due to our comprehensive test coverage and the number of tests that are run. While the verbosity of our output can be useful in certain situations, it also has the potential to generate a significant amount of noise, which can make it difficult to quickly identify and understand failing test cases.

By removing the -v flag, our test output will become more focused. Instead of printing the results of every individual test, the go test command will only print the results of failing tests. This will make our test logs cleaner and easier to navigate, accelerating the process of locating and debugging failing tests.

This change should not impact the effectiveness of our tests or our ability to debug them when they fail. It will simply declutter our test logs and make it easier to spot the important information. I believe this change will benefit our development process and enhance our efficiency.

@MaxMustermann2 MaxMustermann2 requested a review from Frozen August 3, 2023 13:17
@ONECasey ONECasey merged commit fa9546d into harmony-one:dev Aug 4, 2023
@diego1q2w diego1q2w deleted the disable-go-test-v-dev branch August 4, 2023 09:20
@diego1q2w diego1q2w restored the disable-go-test-v-dev branch August 4, 2023 15:10
@diego1q2w diego1q2w deleted the disable-go-test-v-dev branch August 4, 2023 15:12
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