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

Raise PR against master for customer-requested changes #892

Open
sarunnaik opened this issue Sep 17, 2024 · 0 comments
Open

Raise PR against master for customer-requested changes #892

sarunnaik opened this issue Sep 17, 2024 · 0 comments

Comments

@sarunnaik
Copy link

Hi!

I work for @qnx and have recently been looking into supporting cJSON on QNX 7.1 and 8.0 after some of our customers requested it. Changes that enable cross-compiling cJSON for QNX are only a few lines long and confined to files CMakeLists.txt and tests/CMakeLists.txt. Build files reside separately in QNX's GitHub repository for ports.

I noticed that, per the contributing guidelines, pull requests should typically be raised against the develop branch. However, the develop branch seems to be many commits behind master, and given the urgent need of these changes, I would request permission to raise a PR directly to the master branch.

Why a PR against master?

  • The develop branch is many commits behind master, and merging to develop could delay the deployment of these updates.
  • These changes are customer-driven and would benefit from being available in production soon.

I am happy to follow any additional steps you recommend and will be available for any required revisions to the PR :)

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

No branches or pull requests

1 participant