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 2455]: Add a "FAQ for New Contributors" section to the CONTRIBUTING.md file #2091

Open
wants to merge 7 commits into
base: 2.x
Choose a base branch
from

Conversation

harshittiwariii
Copy link

#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.

@Skyllarr
Copy link
Contributor

Skyllarr commented Feb 5, 2024

@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!

@Skyllarr
Copy link
Contributor

Skyllarr commented Feb 6, 2024

@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

@harshittiwariii
Copy link
Author

@Skyllarr Squashed commits and force pushed to branch.
Thank you.

@harshittiwariii
Copy link
Author

@Skyllarr Can you please review and merge this request.

@darranl darranl added the +1 DAL label Mar 8, 2024
@fjuma
Copy link
Contributor

fjuma commented Mar 8, 2024

@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!

@harshittiwariii
Copy link
Author

@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.

@harshittiwariii
Copy link
Author

harshittiwariii commented Mar 11, 2024

@fjuma @darranl I think the conflicts has been resolved. I have pushed the changes to this PR

@Skyllarr
Copy link
Contributor

Hi @harshittiwariii , there is still a conflict in fhe CONTRIBUTING.md file

@harshittiwariii
Copy link
Author

Hi @harshittiwariii , there is still a conflict in fhe CONTRIBUTING.md file

@Skyllarr Can you verify now ?

@Skyllarr
Copy link
Contributor

Conflict was resolved, thank you @harshittiwariii . Please make sure that this PR contains a single commit, currently there are 7 commits

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