Skip to content

fix: Fixing issue with bump_type not working and version name not defaulting#14

Merged
igorlamos merged 3 commits into
beplus:masterfrom
jgsheffer:master
Mar 7, 2022
Merged

fix: Fixing issue with bump_type not working and version name not defaulting#14
igorlamos merged 3 commits into
beplus:masterfrom
jgsheffer:master

Conversation

@jgsheffer

Copy link
Copy Markdown
Contributor

Context

When no version name was passed in it would fill the version name as ""
bump_type was only being used when no version name was passed in

Changes

  • If no version name is passed in it will continue to use the expected default path to look up the version name
  • If valid bump_type is passed in it will increase the version name
  • Added additional tests

Updating to use default gradle path and bump_type
Adding tests for bump and default gradle version
@igorlamos

Copy link
Copy Markdown
Contributor

@jgsheffer shouldn’t we update the version in lib/fastlane/plugin/versioning_android/version.rb before merging? Thanks!

@jgsheffer

Copy link
Copy Markdown
Contributor Author

@jgsheffer shouldn’t we update the version in lib/fastlane/plugin/versioning_android/version.rb before merging? Thanks!

Good callout! I just got that pushed in.

@igorlamos igorlamos merged commit 5a7eefd into beplus:master Mar 7, 2022
@igorlamos

Copy link
Copy Markdown
Contributor

@jgsheffer PR was merged ... I had to migrate the build job on Travis CI and it's failing now. Let me check & fix that and release v0.1.1. Thanks!

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.

2 participants