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

Split the bundle creation steps for release and code update. #2394

Merged
merged 3 commits into from
Jul 15, 2020

Conversation

SergioEstevao
Copy link
Contributor

Change the bundle step to only create updated bundles and not get
updates strings from the web.
This way for code updates only we just need to run yarn bundle.
For the releases, we run yarn release:bundle.
This will accelerate the builds for WP-Android and for WP-iOS it will
allow deploy and test of PR that need bundle updates quicker.

To test:

  • Run yarn bundle and check that only the bundles are updated with local string
  • Run yarn release:bundle check that bundles are updated and we also get new strings that come from the web repo.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Change the bundle step to only create updated bundles and not get
updates strings from the web.
This way for code updates only we just need to run yarn bundle.
For the releases we run yarn release:bundle.
This will accelerate the builds for WP-Android and for WP-iOS it will
allow deploy and test of PR that need bundle updates quicker.
@SergioEstevao SergioEstevao added this to the 1.31 milestone Jun 17, 2020
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jun 17, 2020

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@mchowning
Copy link
Contributor

This generally looks good to me. Nice idea! 👍

My only concern is that I think it could be a bit misleading now that yarn bundle no longer does everything it used to. What would you think about keeping bundle's behavior unchanged (or giving it a new name) and instead adding a new script (i.e., jsBundle) for just generating the JS bundles? Basically, I'm concerned that it would be easy to miss that releases need release:bundle instead of bundle.

@SergioEstevao
Copy link
Contributor Author

My only concern is that I think it could be a bit misleading now that yarn bundle no longer does everything it used to. What would you think about keeping bundle's behavior unchanged (or giving it a new name) and instead adding a new script (i.e., jsBundle) for just generating the JS bundles? Basically, I'm concerned that it would be easy to miss that releases need release:bundle instead of bundle.

That is a good call, do you think we should update the regular WP-Android build to use jsBuild? or that can be problematic when doing production builds for Android?

@mchowning
Copy link
Contributor

That is a good call, do you think we should update the regular WP-Android build to use jsBuild? or that can be problematic when doing production builds for Android?

If we stay with the current no-jitpack build I think it would be a really nice speedup to development to make that change on WPAndroid. It's a slightly tricky change though (since we need to make sure that both CI builds and local release builds generated the full bundle), and it looks like there is a good chance we'll replace our current no-jitpack build with a bintray-based solution, so for the time being I would prefer to just leave things as they are on WPAndroid (assuming we end up not changing the behavior of bundle).

@mchowning
Copy link
Contributor

mchowning commented Jun 18, 2020

👋 @SergioEstevao ! We're close to merging the monorepo changes 🎉 , and we will be cutting the release branches as soon as the monorepo changes are merged to gutenberg-mobile. As a result, we may be cutting the 1.31 release branches earlier than normal.

It looks like these changes can wait until 1.32, so I'm going to bump this to the 1.32 milestone. Please don't merge this until after the 1.31 release is cut because we don't want to add more changes to the 1.31 release than are necessary—this release is already going to have a lot more changes than a normal release because of the monorepo changes.

Of course, if it is important that this PR make the 1.31 release, go ahead and move it back to the 1.31 milestone and let me know so I can try to help get this merged before we cut the release. 🙇

@mchowning mchowning modified the milestones: 1.31, 1.32 Jun 18, 2020
@mchowning
Copy link
Contributor

Bumping this to the 1.33 milestone since we're getting ready to cut the 1.32 branches.

With that said, this PR looks good, so it's got the 👍 to merge.

@mchowning mchowning modified the milestones: 1.32, 1.33 Jul 7, 2020
@SergioEstevao SergioEstevao merged commit 3588f96 into develop Jul 15, 2020
@SergioEstevao SergioEstevao deleted the issue/split_bundle_creation_for_release branch July 15, 2020 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants