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

Bump patch version #587

Merged
merged 1 commit into from
Jul 18, 2018
Merged

Bump patch version #587

merged 1 commit into from
Jul 18, 2018

Conversation

RyanJarv
Copy link
Contributor

Skipping 12 because it was used for some ad-hoc testing, also 13 == good luck

Skipping 12 because it was used for some ad-hoc testing, also 13 == good luck
@RyanJarv RyanJarv requested review from cezaraugusto, mbacchi and bbondy and removed request for cezaraugusto July 18, 2018 17:21
@RyanJarv
Copy link
Contributor Author

@bbondy this will likely be the one we use for the unofficial dev rerelease. Let me know if you have any thoughts here.

Copy link
Contributor

@mbacchi mbacchi left a comment

Choose a reason for hiding this comment

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

+1 (or is it +2 in this case? lol)

RyanJarv added a commit to brave/brave-core that referenced this pull request Jul 18, 2018
RyanJarv added a commit to brave/brave-core that referenced this pull request Jul 18, 2018
Bad things happen if this doesn't match brave/brave-browser#587 and the package.json for this repo.
@RyanJarv
Copy link
Contributor Author

RyanJarv commented Jul 18, 2018

FYI we have one other place we need to update versions in now as well :/

https://github.com/brave/brave-core/pull/268/files#diff-6faf6f9a67e6c736e3a64ad100c7dcdc

If we forget that and don't catch it before release we will get stuck on the chromium versioning scheme. Ticket open for this here: #579

@RyanJarv RyanJarv merged commit 42b70d0 into master Jul 18, 2018
RyanJarv added a commit to brave/brave-core that referenced this pull request Jul 18, 2018
* Bump minor version

Updating to match brave/brave-browser/pull/587

* Bump patch version

Bad things happen if this doesn't match brave/brave-browser#587 and the package.json for this repo.
@cezaraugusto cezaraugusto deleted the RyanJarv-patch-2 branch July 19, 2018 13:55
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