Skip to content

Conversation

@Widcket
Copy link
Contributor

@Widcket Widcket commented Aug 23, 2021

Changes

This PR adds support for the new release automation CLI.

Previously, we were using a custom Fastlane plugin for bumping the version number and updating the Changelog. To bump the version, that plugin uses the IncrementVersionNumberAction action from Fastlane. That action uses the agvtool new-marketing-version command to bump the version number. That command will bump the version number in every Info.plist file it can find, regardless of whether it's actually necessary or not:

Screen Shot 2021-08-23 at 16 43 33

See a previous release of this SDK:

Screen Shot 2021-08-23 at 16 25 57

It's bumping the version number of the test target, even though it's not necessary; only the library target's version number needs to be bumped (JWTDecode/Info.plist).

So this PR sets up the version number replacement for JWTDecode/Info.plist in the .shiprc file, and sets the version number of the test target to be 0.1.0. That means that going forward, each release PR will only be bumping the version number in JWTDecode/Info.plist.

Additionally, this PR removes the now-unused Fastlane task.

Testing

[ ] This change adds unit test coverage (or why not)
[ ] This change has been tested on the latest version of the platform/language or why not

Checklist

[ ] I have read the Auth0 general contribution guidelines
[ ] I have read the Auth0 Code of Conduct
[ ] All existing and new tests complete without errors

@Widcket Widcket requested a review from a team as a code owner August 23, 2021 19:49
@Widcket Widcket added the review:tiny Tiny review label Aug 23, 2021
@Widcket Widcket changed the title Chore/use ship Use the new release automation solution Aug 23, 2021
@Widcket Widcket merged commit 9f2288a into master Aug 24, 2021
@Widcket Widcket deleted the chore/use-ship branch August 24, 2021 11:02
@Widcket Widcket mentioned this pull request Sep 2, 2021
5 tasks
kaishin pushed a commit to kaishin/jwt-decode that referenced this pull request Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review:tiny Tiny review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants