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 2455]: Add a "FAQ for New Contributors" section to the CONTRIBUTING.md file #2091
base: 2.x
Are you sure you want to change the base?
Conversation
@harshittiwariii Thank you for the updates! You can close your old PR if this is the most recent one. But when making updates to your PRs, you can just force push your changes to your branch and then you don't have to open another PR. This PR currently has 5 commits. Please squash them into a single commit. Thanks! |
@harshittiwariii Let us know if you need help with squashing of commits. You can watch our video for guidance on squashing commits: https://www.youtube.com/watch?v=GM9hAfyV5MI |
5448482
to
78356c0
Compare
@Skyllarr Squashed commits and force pushed to branch. |
@Skyllarr Can you please review and merge this request. |
@harshittiwariii Welcome to the WildFly Elytron project, thanks very much for your contribution, it looks good! There's just a conflict that needs to be resolved and then we can merge this. Would you be able to rebase to resolve the conflict? If you have any questions on how to do that, feel free to ask. Thanks! |
@fjuma Thank you for the review. Sure, I will fix the issue. |
78356c0
to
280adfe
Compare
Hi @harshittiwariii , there is still a conflict in fhe CONTRIBUTING.md file |
@Skyllarr Can you verify now ? |
Conflict was resolved, thank you @harshittiwariii . Please make sure that this PR contains a single commit, currently there are 7 commits |
#2090
Issue: https://issues.redhat.com/browse/ELY-2455
Follow up:
-Added link to FAQ's section for New contributors.
-Added link to community documentation.