Skip to content

Fix manifest version bump script#8715

Merged
Gudahtt merged 1 commit intodevelopfrom
fix-manifest-version-bump-script
Jun 1, 2020
Merged

Fix manifest version bump script#8715
Gudahtt merged 1 commit intodevelopfrom
fix-manifest-version-bump-script

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Jun 1, 2020

The version bump script referenced the old file path for the manifest. It was stored as a single file called manifest.json, but it was split into _base.json and platform-specific manifests in #8140.

The manifest version bump script has been updated to reference _base.json, which is the piece that has the version property.

The version bump script referenced the old file path for the manifest.
It was stored as a single file called `manifest.json`, but it was split
into `_base.json` and platform-specific manifests in #8140.

The manifest version bump script has been updated to reference
`_base.json`, which is the piece that has the version property.
@Gudahtt Gudahtt requested review from a team and kumavis as code owners June 1, 2020 20:20
@Gudahtt
Copy link
Copy Markdown
Member Author

Gudahtt commented Jun 1, 2020

I tested this locally by commenting out the last line (the git push), and running it with the expected CI environment variables set, and a fake version:

CI=true CIRCLECI=true CIRCLE_BRANCH=Version-v10.0.0 bash ./.circleci/scripts/release-bump-manifest-version

Copy link
Copy Markdown
Contributor

@whymarrh whymarrh left a comment

Choose a reason for hiding this comment

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

LGTM!

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [9706c40]
Page Load Metrics (656 ± 57 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint318645178
domContentLoaded39079265411857
load39279465611857
domInteractive39079265411857

@Gudahtt Gudahtt merged commit 991ef36 into develop Jun 1, 2020
@Gudahtt Gudahtt deleted the fix-manifest-version-bump-script branch June 1, 2020 21:07
Gudahtt added a commit that referenced this pull request Jun 1, 2020
* origin/develop:
  Fix manifest version bump script (#8715)
  Consistent rounded corners on tx details modal/popup (#8714)
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.

4 participants