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

chore: increase version number to 0.16.24 #2170

Merged
merged 3 commits into from
Jan 30, 2024

Conversation

esmeetewinkel
Copy link
Collaborator

PR Checklist

  • PR title descriptive (can be used in release notes)

Description

PR increase version number to 0.16.24

@github-actions github-actions bot added the maintenance Core updates, refactoring and code quality improvements label Dec 15, 2023
@github-actions github-actions bot added maintenance Core updates, refactoring and code quality improvements and removed maintenance Core updates, refactoring and code quality improvements labels Dec 15, 2023
@@ -129,7 +129,7 @@ jobs:
echo "Version Code: $VERSION_CODE"
echo "Version: $VERSION"
# This will need to change currently looking for specific version and not place holder
sed -i -e "s/16022/$VERSION_CODE/g" -e "s/0.16.22/$VERSION/g" ./android/app/build.gradle
Copy link
Collaborator Author

@esmeetewinkel esmeetewinkel Dec 15, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jfmcquade Should this be added to the script yarn scripts version? This is what I normally use to bump the version but it only updates the other two places...

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. Really we want to action #2136, but this is probably a good idea in the short term, I'll have a look

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we're updating the android build number for content repos shouldn't we be using a variable coming from the content repo itself? E.g. the content repo version number...

Copy link
Collaborator

@jfmcquade jfmcquade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've also bumped the version number in the github workflow reusable-android-build.yml, which is used by content repos. Currently it's still looking for a specific version number (now increased to match the other files) to replace with the target version number specified by the repo that is invoking the action. This is a workaround awaiting fixing #2136

@github-actions github-actions bot added maintenance Core updates, refactoring and code quality improvements and removed maintenance Core updates, refactoring and code quality improvements labels Jan 30, 2024
@jfmcquade jfmcquade merged commit ed09cf0 into master Jan 30, 2024
8 checks passed
@jfmcquade jfmcquade deleted the chore/increase-version-0.16.24 branch January 30, 2024 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Core updates, refactoring and code quality improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants