Skip to content

Remove script for creating master sync PR#10791

Merged
Gudahtt merged 1 commit intodevelopfrom
remove-master-sync-script
Mar 31, 2021
Merged

Remove script for creating master sync PR#10791
Gudahtt merged 1 commit intodevelopfrom
remove-master-sync-script

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Mar 31, 2021

The script responsible for creating the "Sync master with develop" PR has been removed. We will soon be eliminating the need for a master branch altogether, so we don't need this anymore. Also, this script hasn't been running correctly in a long time. We've been creating this PR manually.

The script responsible for creating the "Sync `master` with `develop`"
PR has been removed. We will soon be eliminating the need for a
`master` branch altogether, so we don't need this anymore. Also, this
script hasn't been running correctly in a long time. We've been
creating this PR manually.
@Gudahtt Gudahtt requested review from a team and kumavis as code owners March 31, 2021 22:06
@Gudahtt Gudahtt requested a review from shanejonas March 31, 2021 22:06
Copy link
Copy Markdown
Contributor

@ryanml ryanml left a comment

Choose a reason for hiding this comment

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

++

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [bd00b59]
Page Load Metrics (644 ± 57 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint52766273
domContentLoaded36678264311857
load36878364411857
domInteractive36678264211857

@Gudahtt Gudahtt merged commit 358fec0 into develop Mar 31, 2021
@Gudahtt Gudahtt deleted the remove-master-sync-script branch March 31, 2021 22:24
@github-actions github-actions bot locked and limited conversation to collaborators Mar 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants