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

Add a link to the project status page on CircleCI #63

Merged
merged 2 commits into from
Sep 20, 2019

Conversation

tnir
Copy link
Contributor

@tnir tnir commented Aug 29, 2019

  • Avoids to use shields.io's badge
  • Adds a useful link to the CircleCI pipeline results

cf. some usecase in a certain project: wbernest/mattermost-plugin-rssfeed#18

@hanzei hanzei self-requested a review August 31, 2019 16:15
@hanzei hanzei added the 2: Dev Review Requires review by a core committer label Aug 31, 2019
README.md Outdated Show resolved Hide resolved
@mattermod
Copy link
Contributor

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

/cc @jasonblais @hanzei

@hanzei
Copy link
Contributor

hanzei commented Sep 19, 2019

@tnir If you don't mind I will do the proposed changes and then merge.

Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
@tnir tnir changed the title Use the CircleCI official status badge with a link Add a link to the project status page on CircleCI Sep 19, 2019
@hanzei
Copy link
Contributor

hanzei commented Sep 19, 2019

@tnir Please merge master into your branch

Copy link
Contributor

@hanzei hanzei 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 👍

I have one small suggestion.

README.md Outdated Show resolved Hide resolved
Co-Authored-By: Ben Schumacher <ben.schumacher@mattermost.com>
@tnir
Copy link
Contributor Author

tnir commented Sep 20, 2019

@hanzei Thanks for the comment. Adpated thr suggestion.

@alifarooq0
Copy link

@tnir if you merge master into your branch that should fix the ci error

@hanzei hanzei added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels Sep 20, 2019
@hanzei hanzei merged commit 69b6ae8 into mattermost:master Sep 20, 2019
@tnir tnir deleted the better-circleci-badge branch September 20, 2019 07:54
@tnir
Copy link
Contributor Author

tnir commented Sep 20, 2019

Thanks @ali-farooq0

Was able to confirm all tests on CI passed at https://github.com/mattermost/mattermost-plugin-starter-template/runs/229404880

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants