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 Contributors and Amendments sections #6

Merged
merged 2 commits into from
Aug 31, 2018
Merged

Add Contributors and Amendments sections #6

merged 2 commits into from
Aug 31, 2018

Conversation

fdiaz
Copy link
Collaborator

@fdiaz fdiaz commented Aug 31, 2018

Summary

Add Contributors and Amendments sections

Reasoning

Following our Javascript Style Guide we're adding a section for contributors and a section for amendments

Reviewers

cc @airbnb/swift-styleguide-maintainers


We encourage you to fork this guide and change the rules to fit your team’s style guide. Below, you may list some amendments to the style guide. This allows you to periodically update your style guide without having to deal with merge conflicts.

**[⬆ back to top](#table-of-contents)**
Copy link
Contributor

Choose a reason for hiding this comment

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

Let’s add a trailing new line

@fdiaz fdiaz merged commit 4a96020 into master Aug 31, 2018
@fdiaz fdiaz deleted the fd/amendments branch August 31, 2018 17:59
@dcramps
Copy link
Collaborator

dcramps commented Aug 31, 2018

@fdiaz on mobile so can’t open a fix PR, but contributors links to the JavaScript graph. Missed this in my review :(

@fdiaz
Copy link
Collaborator Author

fdiaz commented Aug 31, 2018

Lol, nice catch! Doing another PR to fix.


## Contributors

- [View Contributors](https://github.com/airbnb/javascript/graphs/contributors)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This links to our JS repo

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yup! #6 (comment) and #12

My bad! 😁

@fdiaz
Copy link
Collaborator Author

fdiaz commented Aug 31, 2018

Fixed here #12

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.

4 participants