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.

Sign up for GitHub

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

Jump to bottom

tests: remove echo=False [backport 2018.07] #9750

Merged
merged 5 commits into from
Aug 9, 2018
Merged

tests: remove echo=False [backport 2018.07] #9750

merged 5 commits into from
Aug 9, 2018

Conversation

cladmi
Copy link
Contributor

@cladmi cladmi commented Aug 9, 2018

Backport of #9744

Contribution description

All tests should be run with an output by default.

Issues/PRs references

No output for successful tests when building tests in native in release tests.

RIOT-OS/Release-Specs#69 (comment)

@cladmi cladmi added Process: release backport Integration Process: The PR is a release backport of a change previously provided to master CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: tests Area: tests and testing framework labels Aug 9, 2018
@cladmi cladmi requested a review from miri64 August 9, 2018 10:23
This was referenced Aug 9, 2018
Merged
Closed
miri64
miri64 reviewed Aug 9, 2018
View reviewed changes
Copy link
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

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

ACK

@cladmi
Copy link
Contributor Author

cladmi commented Aug 9, 2018

@miri64 Can you do a Review 'Approve' and merge ? :)

miri64
miri64 approved these changes Aug 9, 2018
View reviewed changes
Copy link
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

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

ACK

@miri64 miri64 merged commit bf44370 into RIOT-OS:2018.07-branch Aug 9, 2018
@cladmi cladmi deleted the 2018.07/pr/tests/run_all_with_echo_true branch August 9, 2018 15:04
@cladmi cladmi added this to the Release 2018.07 milestone Aug 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miri64 miri64 miri64 approved these changes

Assignees
No one assigned
Labels
Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master
Projects
None yet
Milestone
Release 2018.07
Development

Successfully merging this pull request may close these issues.
2 participants
@cladmi @miri64