hoodiehq / camp

:circus_tent: Welcome to Hoodie Camp!
https://hoodie.camp
Apache License 2.0
99 stars 55 forks source link

Correct link of code coverage badge in hoodie-account-server-api’s README #97

Closed LowProfileDog closed 7 years ago

LowProfileDog commented 7 years ago

πŸ†•πŸ₯☝ First Timers Only

This issue is reserved for people who never contributed to Open Source before. We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you πŸ’ About First Timers Only.

πŸ€” What you will need to know

Nothing. This issue is meant to welcome you to Open Source :) We are happy to walk you trough the process

❓ The Motivation

This is hoodie-account-server-api’s code coverage badge: Coverage Status. You can see it on top of hoodie-account-server-api’s README.

The badge is linked to the coverage detail page. The URL of this page recently changed. The old URL is https://coveralls.io/r/hoodiehq/hoodie-account-server-api?branch=master while the new one is https://coveralls.io/github/hoodiehq/hoodie-account-server-api?branch=master. The user is currently redirected but we don’t know for how long, so better to update it to the new URL.

:clipboard: Step by Step

πŸ€”β“ Questions

Best is to ask @polinadotio who is at the event herself. You can also ping us in the Hoodie Chat or on Twitter

shaduk commented 7 years ago

Hi. This would be my first PR. Please allow me to collaborate on this issue. Thanks for making open source easy for first timers. :)

LowProfileDog commented 7 years ago

welcome @shaduk πŸ‘‹

The issue is all yours :) I’ve invited you to our Camp repository, you can accept my invitation at https://github.com/hoodiehq/camp/invitations

Once accepted, you can assign yourself to this issue and check off the checkboxes above. Enjoy πŸ•

shaduk commented 7 years ago

Hello @LowProfileDog Please review the changes I have done. Is there anything else I need to do?. Thanks for your support.

gr2m commented 7 years ago

it’s not yet done :) This issue gets closed once we merge your pull request, I’ll have a look now