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 LGTM code quality badges #640

Merged
merged 3 commits into from Jan 18, 2019
Merged

Add LGTM code quality badges #640

merged 3 commits into from Jan 18, 2019

Conversation

xcorail
Copy link
Contributor

@xcorail xcorail commented Sep 5, 2018

Hi there!

I thought you might be interested in adding these code quality badges to your project. They will indicate a very high code quality to potential users and contributors.
To get an idea of the analyses reflected by these grades, check the alerts discovered by LGTM.

N.B.: I am on the team behind LGTM.com, I'd appreciate your feedback on this initiative, whether you're interested or not, if you find time to drop me a line. Thanks.

@Siemienik Siemienik merged commit 82b7305 into exceljs:master Jan 18, 2019
@xcorail
Copy link
Contributor Author

xcorail commented Jan 18, 2019

Hello @Siemienik

Happy you found it useful.
A few precisions on the runs:

  • Currently, LGTM polls your repo once a day, and analyzes all new commits, most recent first
  • We recommend to activate Automated Code Review. This way LGTM analyzes your Pull Requests immediately and report any new problem directly in the comments. It's free, and it's a one click operation

For support, questions, feedback: https://discuss.lgtm.com/
Cheers

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.

None yet

2 participants