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

Enable Report Card. #56

Merged
merged 1 commit into from
Mar 25, 2016
Merged

Conversation

nathanleiby
Copy link
Contributor

Generates a Report Card of how well your repo meets Clever's best practices.
Its result is visible via GH Status hook, and you can see all details by looking in Drone's output.

For context, see https://clever.atlassian.net/browse/INFRA-1577.


This PR was autogenerated. You were assigned this PR as the most active committer to this repo (or because @nathanleiby chose you). Congrats!

@@ -14,6 +17,7 @@ publish:
- sphinx-amd64.deb
repo: sphinx
script:
- sudo pip install -q git+https://$REPORT_CARD_GITHUB_REPO_TOKEN@github.com/Clever/report-card.git; GITHUB_API_TOKEN=$REPORT_CARD_GITHUB_STATUS_TOKEN report-card --publish || true
Copy link
Contributor

Choose a reason for hiding this comment

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

indentation off

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Looks like this is a mistake -- this line be removed

@Christopher-Bradshaw
Copy link
Contributor

build failing Error: missing Docker image

Runs report-card in drone.
Sets two env vars, used to pip install report-card and update GH status.
@nathanleiby
Copy link
Contributor Author

@Christopher-Bradshaw I removed the accidentally inserted line and rebuilt.. looks good now!

@Christopher-Bradshaw Christopher-Bradshaw merged commit bb1890d into master Mar 25, 2016
@Christopher-Bradshaw Christopher-Bradshaw deleted the INFRA-1577-add-report-card branch March 25, 2016 21:36
@Christopher-Bradshaw
Copy link
Contributor

It does! merging.

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.

3 participants