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

Added Xcode 16 adaptation for version 2.2.2. #138

Merged
merged 10 commits into from
Oct 3, 2024

Conversation

kimbely0320
Copy link
Contributor

@kimbely0320 kimbely0320 commented Oct 3, 2024

Description

Added Xcode 16 adaptation for version 2.2.2.

Change Notes

  • Added #include <stdint.h> for RapidParserNewTagCallbacks.h & RapidParserState.h & RapidParserStateHandlers.h

Pre-submission Checklist

  • I ran the unit tests locally before checking in.
  • I made sure there were no compiler warnings before checking in.
  • I have written useful documentation for all public code.
  • I have written unit tests for this new feature.

@kimbely0320 kimbely0320 mentioned this pull request Oct 3, 2024
@CLAassistant
Copy link

CLAassistant commented Oct 3, 2024

CLA assistant check
All committers have signed the CLA.

@kimbely0320 kimbely0320 force-pushed the release/2.2.2_xcode16 branch from 8fbce3f to 6b412fb Compare October 3, 2024 03:30
@theRealRobG
Copy link
Collaborator

Hey, sorry I just noticed the issue you opened 2 weeks ago.

The changes you're looking to apply have already been merged in this PR: #126

It seems that the branch you are working from is just several commits behind (35 actually). I believe that if I merge the latest changes from develop into your branch the only remaining change will be the version bump in the podspec file. Are you OK for me to try merging back up and seeing the result?

Also, there are a few more locations to bump the version, particularly the MARKETING_VERSION values within the xcodeproj. I know the version bumping within the files has been fairly lackluster, but I'm looking to improve this, and hopefully the release process too.

Perhaps this command (at the root of the repo) can make the version bump easier:
For macOS:

sed -i '' "s/MARKETING_VERSION = [0-9]\{1,\}\.[0-9]\{1,\}\.[0-9]\{1,\};/MARKETING_VERSION = 2.2.3;/" ./mamba.xcodeproj/project.pbxproj

For Linux:

sed -i "s/MARKETING_VERSION = [0-9]\{1,\}\.[0-9]\{1,\}\.[0-9]\{1,\};/MARKETING_VERSION = 2.2.3;/" ./mamba.xcodeproj/project.pbxproj

Difference explained: https://stackoverflow.com/a/4247319

Are you ultimately looking for an official release tag update (to 2.2.3) with the compilation fixes?

@kimbely0320
Copy link
Contributor Author

kimbely0320 commented Oct 3, 2024 via email

@theRealRobG
Copy link
Collaborator

No problem.

I just merged and the changelog is now showing that the change is just the version bump to the podspec.

Could you bump all of the MARKETING_VERSION values in ./mamba.xcodeproj/project.pbxproj?

After that I can merge and I should be able to create a release tag from there (will likely have to wait until tomorrow, US Eastern, just to double-check with the other maintainer on the correct process).

Thanks for the contribution and patience!

@kimbely0320
Copy link
Contributor Author

kimbely0320 commented Oct 3, 2024 via email

@theRealRobG theRealRobG merged commit 0fe96b4 into Comcast:develop Oct 3, 2024
4 checks passed
@theRealRobG theRealRobG mentioned this pull request Oct 3, 2024
4 tasks
@theRealRobG
Copy link
Collaborator

Hey @kimbely0320 , I think we're actually going to release this change as part of the 2.3.0 version here: #140

There have been a few other additions since 2.2.2 so I believe a minor bump is more accurate than a patch.

@kimbely0320
Copy link
Contributor Author

kimbely0320 commented Oct 3, 2024 via email

@theRealRobG
Copy link
Collaborator

@kimbely0320 the changes are now available in release 2.3.0.

Thanks for the patience.

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

Successfully merging this pull request may close these issues.

5 participants