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

fix: using a custom badge template breaks badge replacement #210

Merged
merged 2 commits into from
Oct 19, 2019

Conversation

kharaone
Copy link
Contributor

@kharaone kharaone commented Oct 16, 2019

What:
Fixes #120

Why:
Some repositories want to use HTML templates or custom templates for the badges

How:
changed the check done through an hardcoded regex to a simple "is it between these 2 tags"

Checklist:

  • Documentation
  • Tests
  • Ready to be merged

Credits to @simbas
docs are done through all-contributors/all-contributors#312

kharaone added a commit to kharaone/all-contributors that referenced this pull request Oct 17, 2019
Update of the documentation for all-contributors/cli#210, we update
@kharaone kharaone marked this pull request as ready for review October 17, 2019 17:07
@Berkmann18 Berkmann18 changed the title fix: Using a custom badge template breaks badge replacement fix: using a custom badge template breaks badge replacement Oct 18, 2019
@Berkmann18 Berkmann18 merged commit 72de75a into all-contributors:master Oct 19, 2019
@Berkmann18
Copy link
Member

@all-contributors Please add @kharaone for code

@allcontributors
Copy link
Contributor

@Berkmann18

I've put up a pull request to add @kharaone! 🎉

Berkmann18 pushed a commit to all-contributors/all-contributors that referenced this pull request Oct 19, 2019
@all-contributors-release-bot
Copy link
Member

🎉 This PR is included in version 6.9.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@jakebolam
Copy link
Collaborator

I think this was a breaking change.

All Contributors badge will no longer be updated unless they wrap their badge in the following tags

<!-- ALL-CONTRIBUTORS-BADGE:START - Do not remove or modify this section -->
BADGE
<!-- ALL-CONTRIBUTORS-BADGE:END --> 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using a custom badge template breaks badge replacement
4 participants