Skip to content

feat: Add ability to add Version manually#3

Merged
Grohden merged 3 commits into
Grohden:masterfrom
Euganox:patch-1
Sep 7, 2020
Merged

feat: Add ability to add Version manually#3
Grohden merged 3 commits into
Grohden:masterfrom
Euganox:patch-1

Conversation

@andreieuganox

Copy link
Copy Markdown
Contributor

User Story
As a developer I want to be able to define the Version manually.

Background
If for example you use CLI on CI level, you probably use Semantic Release (https://www.npmjs.com/package/semantic-release) to define the Versions of the Application. In this case, the versions created by react-native-cli-bump-version and Semantic Release will be different. To be able to use it with SR it will be good to add additional parameter to this CLI.

P.S. The changes were not tested tbh...sorry for that!
P.S.S. Thank you for your work!

User Story:
As a developer I want to be able to define the Version manually.

Background:
If for example you use CLI on CI level, you probably use Semantic Release (https://www.npmjs.com/package/semantic-release) to define the Versions of the Application. In this case, the versions created by react-native-cli-bump-version and Semantic Release will be different. To be able to use it with SR it will be good to add additional parameter to this CLI
@Grohden

Grohden commented Sep 6, 2020

Copy link
Copy Markdown
Owner

Thanks for the PR! I'm gona review it ASAP

@Grohden Grohden left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

You also need to change the root react-native.config.js to pass this new flag down

Comment thread src/index.ts Outdated
Comment thread src/index.ts Outdated

@andreieuganox andreieuganox left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

good to go

Comment thread src/index.ts Outdated
@Grohden Grohden merged commit 608c154 into Grohden:master Sep 7, 2020
@Grohden

Grohden commented Sep 7, 2020

Copy link
Copy Markdown
Owner

@andreieuganox again, thx for the PR, I'm gonna write tests and soon release it!

@Grohden

Grohden commented Sep 7, 2020

Copy link
Copy Markdown
Owner

1.0.4 released, feel free to ask me anything in case something is wrong

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