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

feat(types): add/update contrib types #24

Merged
merged 7 commits into from
Jan 25, 2017
Merged

feat(types): add/update contrib types #24

merged 7 commits into from
Jan 25, 2017

Conversation

kentcdodds
Copy link
Collaborator

To "upgrade" to the latest specification.

I'm not sure whether to consider this a breaking change...

Kent C. Dodds added 2 commits January 22, 2017 20:17
To "upgrade" to the latest specification.
@kentcdodds
Copy link
Collaborator Author

Rats, I might have to actually pull this down sometime...

itaisteinherz and others added 5 commits January 25, 2017 08:27
* Update format-contribution-type.test.js

Updated the expected question emoji.

* Update format-contributor.test.js

Updated the expected question emoji.

* Add @itaisteinherz as a contributor
* Update README.md

Added the new contribution types to accommodate #24.

* Add @itaisteinherz as a contributor
@@ -88,8 +92,9 @@ These are the keys you can specify:
Thanks goes to these wonderful people ([emoji key](https://github.com/kentcdodds/all-contributors#emoji-key)):

<!-- ALL-CONTRIBUTORS-LIST:START - Do not remove or modify this section -->
| [<img src="https://avatars.githubusercontent.com/u/3869412?v=3" width="100px;"/><br /><sub>Jeroen Engels</sub>](https://github.com/jfmengels)<br />[💻](https://github.com/jfmengels/all-contributors-cli/commits?author=jfmengels) [📖](https://github.com/jfmengels/all-contributors-cli/commits?author=jfmengels) [⚠️](https://github.com/jfmengels/all-contributors-cli/commits?author=jfmengels) | [<img src="https://avatars.githubusercontent.com/u/1500684?v=3" width="100px;"/><br /><sub>Kent C. Dodds</sub>](http://kentcdodds.com/)<br />[📖](https://github.com/jfmengels/all-contributors-cli/commits?author=kentcdodds) | [<img src="https://avatars.githubusercontent.com/u/14871650?v=3" width="100px;"/><br /><sub>João Guimarães</sub>](https://github.com/jccguimaraes)<br />[💻](https://github.com/jfmengels/all-contributors-cli/commits?author=jccguimaraes) | [<img src="https://avatars.githubusercontent.com/u/1282980?v=3" width="100px;"/><br /><sub>Ben Briggs</sub>](http://beneb.info)<br />[💻](https://github.com/jfmengels/all-contributors-cli/commits?author=ben-eb) |
| :---: | :---: | :---: | :---: |
| [<img src="https://avatars.githubusercontent.com/u/3869412?v=3" width="100px;"/><br /><sub>Jeroen Engels</sub>](https://github.com/jfmengels)<br />[💻](https://github.com/jfmengels/all-contributors-cli/commits?author=jfmengels) [📖](https://github.com/jfmengels/all-contributors-cli/commits?author=jfmengels) [⚠️](https://github.com/jfmengels/all-contributors-cli/commits?author=jfmengels) | [<img src="https://avatars.githubusercontent.com/u/1500684?v=3" width="100px;"/><br /><sub>Kent C. Dodds</sub>](http://kentcdodds.com/)<br />[📖](https://github.com/jfmengels/all-contributors-cli/commits?author=kentcdodds) | [<img src="https://avatars.githubusercontent.com/u/14871650?v=3" width="100px;"/><br /><sub>João Guimarães</sub>](https://github.com/jccguimaraes)<br />[💻](https://github.com/jfmengels/all-contributors-cli/commits?author=jccguimaraes) | [<img src="https://avatars.githubusercontent.com/u/1282980?v=3" width="100px;"/><br /><sub>Ben Briggs</sub>](http://beneb.info)<br />[💻](https://github.com/jfmengels/all-contributors-cli/commits?author=ben-eb) | [<img src="https://avatars.githubusercontent.com/u/22768990?v=3" width="100px;"/><br /><sub>Itai Steinherz</sub>](https://github.com/itaisteinherz)<br />[📖](https://github.com/jfmengels/all-contributors-cli/commits?author=itaisteinherz) [💻](https://github.com/jfmengels/all-contributors-cli/commits?author=itaisteinherz) |
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the contributors list got duplicated because of 92e2aa2.

@kentcdodds
Copy link
Collaborator Author

I was hoping that I wouldn't have to actually pull it down (I'm so impressed with how much GitHub's web client is capable of these days!) but I got the formatting wrong on the table :face_palm: so I've pulled it down and regenerated things :) Should be good now I think. Will let @jfmengels merge and release if he thinks it's ready.

@jfmengels jfmengels merged commit 8fd4098 into master Jan 25, 2017
@jfmengels jfmengels deleted the pr/types-upgrade branch January 25, 2017 17:13
@jfmengels
Copy link
Collaborator

Thanks to you both!
@kentcdodds You made this a breaking change in the spec right? Do you think this should be a major release then? I think probably not, but I'm down with whatever.

By the way, I just gave you publishing rights, I trust your judgement in handling this tool, you use it more than I do 😄

@kentcdodds
Copy link
Collaborator Author

Do you mind if I setup semantic-release on it?

@jfmengels
Copy link
Collaborator

No I don't, but I have never learnt how to use it (or rather, what the commit messages should look like). I guess it's time to do so :D

@kentcdodds
Copy link
Collaborator Author

Luckily with GitHub's new "Rebase and Merge" functionality, you can change the commit message of PRs when you merge them right in GitHub's UI which is pretty amazingly awesome. I'll get things setup. Thanks!

Berkmann18 pushed a commit that referenced this pull request May 24, 2020
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