Skip to content

Comments

Improve GitHub status message: show # of changed files#5

Open
wilzbach wants to merge 1 commit intoCyberShadow:masterfrom
wilzbach:improve-github-message
Open

Improve GitHub status message: show # of changed files#5
wilzbach wants to merge 1 commit intoCyberShadow:masterfrom
wilzbach:improve-github-message

Conversation

@wilzbach
Copy link

@wilzbach wilzbach commented Feb 5, 2018

This is something that should be visible on the CI dashboard as it should be immediately a red flag when the number of changed files is high. To save the space required for this, I changed "Documentation OK" to just "OK".

@CyberShadow
Copy link
Owner

CyberShadow commented Feb 5, 2018

Showing # of changed files was something I considered back when designing it... I think in the end the number of lines changed is more important and carries at least as much signal for most circumstances.

Removing the "Documentation" word is probably not a good idea. Including it was also a very deliberate decision. Until you learn what exactly "CyberShadow/DAutoTest" means, you don't know what this CI is doing or which CI is responsible for building the docs.

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