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

[ELY-2236] Updated README.md with link to Contribution guide #1608

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

nimisha-yadav
Copy link
Contributor

@wildfly-ci
Copy link

Hello, nimisha-yadav. I'm waiting for one of the admins to verify this patch with /ok-to-test in a comment.

README.md Outdated
@@ -35,5 +35,6 @@ Contributions
-------------

All new features and enhancements should be submitted to 1.x branch only.

https://wildfly-security.github.io/wildfly-elytron/getting-started-for-developers/
Our [contribution guide](https://github.com/wildfly-security/wildfly-elytron/blob/1.x/CONTRIBUTING.md) will guide you through the steps for getting started on the Elytron Web project and will go through how to format and submit your first PR.
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a small change needed. "Elytron Web project" should be "WildFly Elytron project"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated this

Copy link
Contributor

@fjuma fjuma left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, @nimisha-yadav!

@Skyllarr
Copy link
Contributor

@nimisha-yadav Thank you

@Skyllarr Skyllarr merged commit a6b1368 into wildfly-security:1.x Oct 19, 2021
@nimisha-yadav nimisha-yadav deleted the ELY-2236 branch October 19, 2021 17:45
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.

4 participants