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 Coveralls Community Backend #773

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

merijnS
Copy link

@merijnS merijnS commented Jan 21, 2022

No description provided.

@coveralls
Copy link

coveralls commented Jan 21, 2022

Coverage Status

Coverage increased (+0.0008%) to 97.005% when pulling 32dfe97 on Groninger-Bugbusters:master into a4f099e on chaoss:master.

@vchrombie vchrombie changed the title Add Coveralls Cummunity Backend Add Coveralls Community Backend Jan 22, 2022
@vchrombie
Copy link
Member

vchrombie commented Jan 22, 2022

Thanks for the PR @merijnS. We love to see our contributors come up with more data sources. Thinking about it, Coveralls is an excellent data source and I must appreciate your team @Groninger-Bugbusters have done awesome work.

I have some questions though. Is the backend ready to use? I see that the backend needs a repo URL as an argument but I couldn't see any information regarding it in the readme file.

Also, it would be great if you can fix the issue with the sign-off.

$ git commit --amend -s
$ git push -f origin master

You can also edit the commit message during this time. https://github.com/chaoss/grimoirelab/blob/master/CONTRIBUTING.md#guidelines-to-follow-to-write-good-commit-messages

It can be something like (you can add more details according to the guidelines)

[docs] Add Coveralls Community Backend

This commit adds the coveralls backend repository link to the 
Community Backends section in the README.md file.

<<sign-off string>>

Please let me know if you need any help.

This commit adds the Coveralls backend repository link to the
Community Backends section in the README.md file.

Signed-off-by: Merijn <[email protected]>
@merijnS
Copy link
Author

merijnS commented Jan 27, 2022

Thanks for the info @vchrombie! I have updated the commit.

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

3 participants