Skip to content
This repository has been archived by the owner on Aug 13, 2023. It is now read-only.

Bump babel/cli to 7.8.3 and remove --configFile option from build script #2980

Merged
merged 4 commits into from Jan 22, 2020

Conversation

amywalkerdev
Copy link
Contributor

@amywalkerdev amywalkerdev commented Jan 20, 2020

Overall change:
Bumps @babel/cli from 7.7.0 to 7.8.3 and removes the --configFile option from the package build scripts.

Babel suggests the use of kebabcase in its command line options, and version 7.7.4 of the cli fixes an issue where unknown options would silently fail (babel/babel#10698). I think this means that our use of --configFile babel.config.js has actually been ignored for a while now and it has just been defaulting to our config file in the root anyway. Removing this option, the packages still seem to build as they used to and components look ok in storybook.

Code changes:

  • Bumps the version of @babel/cli to 7.8.3 in package.json and package-lock.json
  • removes the --configFile option from the package build script

  • I have assigned myself to this PR and the corresponding issues
  • Automated jest tests added (for new features) or updated (for existing features)
  • This PR requires manual testing

@amywalkerdev amywalkerdev changed the title Babel cli update Bump babel/cli to 7.8.3 and remove --configFile option from build script Jan 21, 2020
@amywalkerdev amywalkerdev self-assigned this Jan 21, 2020
@amywalkerdev amywalkerdev added this to PR in Progress in Simorgh via automation Jan 21, 2020
@amywalkerdev amywalkerdev added ws-home Tasks for the WS Home Team ws-media The World Service media stream ws-articles Tasks for the WS Articles Team labels Jan 21, 2020
@amywalkerdev amywalkerdev moved this from PR in Progress to Code review in Simorgh Jan 21, 2020
Simorgh automation moved this from Code review to Ready for merge (probably) Jan 22, 2020
@amywalkerdev amywalkerdev merged commit ca20a96 into latest Jan 22, 2020
Simorgh automation moved this from Ready for merge (probably) to Done Jan 22, 2020
@amywalkerdev amywalkerdev deleted the babelcli branch January 22, 2020 09:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ws-articles Tasks for the WS Articles Team ws-home Tasks for the WS Home Team ws-media The World Service media stream
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants