Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Infrastructure: don't print danger message if everything was kosher#6078

Merged
vadi2 merged 2 commits intodevelopmentfrom
dont-print-on-danger
Apr 25, 2022
Merged

Infrastructure: don't print danger message if everything was kosher#6078
vadi2 merged 2 commits intodevelopmentfrom
dont-print-on-danger

Conversation

@vadi2
Copy link
Member

@vadi2 vadi2 commented Apr 25, 2022

Brief overview of PR changes/additions

Don't print danger message if everything was kosher

Motivation for adding to Mudlet

Less noise when you create a PR

Other info (issues closed, discussion etc)

Danger doesn't require printing anything by default: danger/danger-js#1268 (comment)

We'll have to merge this in order to see if it worked.

@add-deployment-links
Copy link

add-deployment-links bot commented Apr 25, 2022

Hey there! Thanks for helping Mudlet improve. 🌟

Test versions

You can directly test the changes here:

No need to install anything - just unzip and run.
Let us know if it works well, and if it doesn't, please give details.

@vadi2 vadi2 merged commit bccaae4 into development Apr 25, 2022
@vadi2 vadi2 deleted the dont-print-on-danger branch April 25, 2022 14:01
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.

2 participants