Discussion space on Pull requests

[Ornithopter-pilot]

We have a lot of PRs coming in, and since I’m the only maintainer here, it’s becoming challenging to review and approve them in a timely manner. As I don’t have a lot of free time, I’m looking to grant roles and access to contributors who are willing to help with this project.

If you’re interested in contributing to the review process or helping manage PRs, please let me know. Your support would be greatly appreciated!

[Ornithopter-pilot]

@bhaumikmaan , I created this space for PR discussions. There are a lot of PRs coming in, and it will take a significant amount of time for me to properly review each one. I can’t manage this in a timely manner on my own.

Since you are actively contributing and have the necessary knowledge, I’m considering granting you access to help with PR reviews. Let me know if you’re interested. We’ll need to be cautious to ensure we maintain the quality and stability of the project.

[bhaumikmaan]

Hey,
I was thinking about something like this so thanks for doing so, sounds like a good initiative.
I am down to help contribute with this and we can scale it a lot more from here on!
Would still bug you for issues that I need opinion on 😛
Thanks

[anshuman018]

Hey,
Really cool to see this coming together — it’s something I had in mind too!
I’m in to contribute and push this further with you.
Might still poke you here and there when I need a second opinion 😄
Cheers!

[Ornithopter-pilot]

@anshuman018 @bhaumikmaan , I have shared you the invite for collaboration and also enforced branch protection role for main . PR requires 2 approval at least and can't be self approved

[Ornithopter-pilot]

I have updated branch protection role to even not allow the admin to push to main . since , we all gonna follow this consistent PR workflow.

[bhaumikmaan]

Yeah cool
I'll take it from here, thinking of adding actions and checks too

[anshuman018]

I think keeping 2 approvals is better for now — it adds an extra layer of review and ensures stability especially since multiple people will now have merge access
We can revisit and adjust it later as things settle down

[Ornithopter-pilot]

Guys , I made the approval required to 1 for PR merge in branch protection rule (it prevents self approval too) . should we keep 1 or should we increase to 2 for safer side? need your thoughts on this

[bhaumikmaan]

Two should be fine for now until we add some prechecks

[Ornithopter-pilot]

Okay , i have updated to 2 from 1 and also , unchecked self approval of PR to prevent the self approval . please tag me , if you guys require anything . again really thanks for the contributions

[Ornithopter-pilot]

If someone need any kind of help , please tag and ask me in this discussion as I will be getting email notification for this discussion .

For PRs or issues, I won't be receiving email notifications, so I will only check them when I log in.

@bhaumikmaan @anshuman018

[anshuman018]

Okayyy

[bhaumikmaan]

ez pz

[bhaumikmaan]

@Ornithopter-pilot
Please add @dev-opsss as contributor. Need active members to ship PR's and issues

[Ornithopter-pilot]

@Ornithopter-pilot Please add @dev-opsss as contributor. Need active members to ship PR's and issues

Added him .

Thanks for your contributions @dev-opsss ❤️