Skip to content

Conversation

@devversion
Copy link
Member

@devversion devversion commented May 24, 2017

Currently the CI logs everything that runs inside of the bash script.

This might have been useful when creating the script but now it really pollutes the CI output which should be clean and structured.

Disabling tracing for the build and test script because the errors will still show up and the CI will look more clean.

Currently the CI traces everything that runs inside of the bash script.

This might have been useful when creating the script but now it really pollutes the CI output which should be clean and structured.

Disabling tracing for the build and test script because the errors will still show up and the CI will look more clean.
@devversion devversion requested a review from jelbourn May 24, 2017 13:46
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label May 24, 2017
Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

@jelbourn jelbourn added pr: lgtm action: merge The PR is ready for merge by the caretaker and removed pr: needs review labels May 24, 2017
@tinayuangao tinayuangao merged commit 426a644 into angular:master May 25, 2017
@devversion devversion deleted the build/silence-ci-scripts branch November 11, 2017 10:20
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants