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 Contributing Guideline #147

Merged
merged 2 commits into from
Mar 8, 2022
Merged

Add Contributing Guideline #147

merged 2 commits into from
Mar 8, 2022

Conversation

Huxpro
Copy link
Contributor

@Huxpro Huxpro commented Mar 8, 2022

As a follow-up of #132 (comment), we should clarify the requirements and expectations on different kinds of contributions.

As a follow-up of <#132 (comment)>, we should clarify the requirements and expectations on different kinds of contributions.
@Huxpro
Copy link
Contributor Author

Huxpro commented Mar 8, 2022

Feel free to comment or open an issue if you have any feedback

@Huxpro Huxpro merged commit a302a08 into main Mar 8, 2022
@Huxpro Huxpro deleted the contributing-guideline branch March 8, 2022 04:01
@@ -104,6 +104,28 @@ Prior Art

The JSX syntax is similar to the [E4X Specification (ECMA-357)](http://www.ecma-international.org/publications/files/ECMA-ST-WITHDRAWN/Ecma-357.pdf). E4X is a deprecated specification with deep reaching semantic meaning. JSX partially overlaps with a tiny subset of the E4X syntax. However, JSX has no relation to the E4X specification.


Contributing
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Make sense. Thanks for the feedback. Will do!

Copy link
Contributor

Choose a reason for hiding this comment

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

Just realized there's already a pending #114 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hah yea I've been thinking of merge them into one ;)

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

Successfully merging this pull request may close these issues.

3 participants