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

v2.5.7 Build Tools Hotfix #1361

Closed
wants to merge 1 commit into from
Closed

v2.5.7 Build Tools Hotfix #1361

wants to merge 1 commit into from

Conversation

sghoweri
Copy link
Contributor

Jira

http://vjira2:8080/browse/BDS-1794

Summary

Hotfix to the Bolt build tools to address a recent Sass Loader config change (when the Webpack build is compiling for production) that started causing recent Bolt builds on Travis to fail.

Details

One of the Bolt build tool dependencies (Sass Loader, used in our Webpack build) very recently introduced a "seemingly small" change that unfortunately has started to cause our Travis builds to start to fail and started failing downstream as reported this morning (likely due to fresher installs / NPM caches expiring and pulling in the more recent Sass Loader version).

Thankfully the fix seems to our Webpack configuration seems to be quite small but still requires a Bolt hotfix to ensure that this issue is patched before any older cached versions of this dependency get updated and builds start to fail across the board.
 
image

image

… to address a recent Sass Loader upstream change + update CLI to force i18n to be enabled by default for prod builds if otherwise undefined
@sghoweri sghoweri changed the title hotfix: update webpack Sass loader config to always output nested CSS… v2.5.7 Build Tools Hotfix Aug 27, 2019
Copy link
Collaborator

@christophersmith262 christophersmith262 left a comment

Choose a reason for hiding this comment

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

Looks pretty safe to me.

@sghoweri
Copy link
Contributor Author

FYI this PR was branched off of the v2.5.6 tag -- hence why a few of the build steps on this PR failed when pointing to the v2.6.0-beta.1 code (on the release branch) but the actual Travis build passed without any issues: https://travis-ci.com/bolt-design-system/bolt/builds/124743235

@sghoweri sghoweri changed the base branch from release/2.x to v2.5.6-tag August 27, 2019 14:22
@sghoweri
Copy link
Contributor Author

Closing this out -- v2.5.7 has been released with this hotfix and v2.6.0-beta.2 (with the same updates) is rolling out the door -- both releases will be announced shortly!

CC @margoromo

@sghoweri sghoweri closed this Aug 27, 2019
@sghoweri sghoweri deleted the v2.5.7 branch August 27, 2019 18:53
sghoweri added a commit that referenced this pull request Aug 27, 2019
… to address a recent Sass Loader upstream change. port of #1361 for the v2.6.0-beta.2 release
sghoweri added a commit that referenced this pull request Aug 27, 2019
Hotfix: Fix Sass Loader in Build Tools (Duplicate of #1361)
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.

None yet

2 participants