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

[Snap-CI] Replace with error message #1468

Merged
merged 2 commits into from
Mar 4, 2018
Merged

[Snap-CI] Replace with error message #1468

merged 2 commits into from
Mar 4, 2018

Conversation

PyvesB
Copy link
Member

@PyvesB PyvesB commented Jan 23, 2018

Hello,

This prepares the work needed to close #1466. There are still discussions around colour in #1449, so I have left out any colour assignments for now. This pull request should therefore probably not be merged until then.

No tests previously existed, so I simply added a "no longer available" test, which will also tick one item from the top table in #1358.

Cheers,

Pyves

@shields-ci
Copy link

shields-ci commented Jan 23, 2018

Messages
📖

✨ Thanks for your contribution to Shields, @PyvesB!

📖

Thanks for contributing to our documentation. We ❤️ our documentarians!

Generated by 🚫 dangerJS

@RedSparr0w
Copy link
Member

Thanks for creating this PR!

Changes look good to me.
Will merge with #1449 once an outcome has been decided.

@paulmelnikow paulmelnikow added the service-badge Accepted and actionable changes, features, and bugs label Jan 24, 2018
Copy link
Member

@platan platan left a comment

Choose a reason for hiding this comment

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

Looks good to me. Well done!

Copy link
Member

@paulmelnikow paulmelnikow left a comment

Choose a reason for hiding this comment

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

Wanna switch this to lightgray and merge it?

@PyvesB PyvesB merged commit 9b56442 into badges:master Mar 4, 2018
@PyvesB
Copy link
Member Author

PyvesB commented Mar 4, 2018

Thanks everyone for reviewing. Merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service-badge Accepted and actionable changes, features, and bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Snap CI is no longer available
5 participants