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

chore: upgrade lingui to 2.8.0 #3893

Merged
merged 4 commits into from
Jun 21, 2019
Merged

chore: upgrade lingui to 2.8.0 #3893

merged 4 commits into from
Jun 21, 2019

Conversation

brandonc
Copy link
Contributor

@brandonc brandonc commented May 17, 2019

Overview

I added a feature to lingui that allows a config file to be specified which really helps out with our externalplugins configuration. Therefore we no longer need to swap out config files when extracting messages.

The practical result is simplified lingui-extract and lingui-compile scripts

Testing

check out companion plugins-private PR https://github.com/mesosphere/dcos-ui-plugins-private/pull/1008

  • npm run util:lingui:extract
  • npm run util:lingui:compile
    (With and without plugins-ee symlink)

Dependencies

I had to rename the .linguirc config file in the plugins-private repo (https://github.com/mesosphere/dcos-ui-plugins-private/pull/1008)

@brandonc brandonc requested a review from natmegs May 17, 2019 08:37
@GeorgiSTodorov
Copy link
Contributor

GeorgiSTodorov commented May 20, 2019

Build is failing with "npm ERR! cipm can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with npm install before continuing."
Edit: I considered pushing my package-lock file, but since I am on Linux there might be some differences or problems.

GeorgiSTodorov
GeorgiSTodorov previously approved these changes May 20, 2019
@brandonc brandonc added the No Pressure Review merge when having some slack label May 23, 2019
@GeorgiSTodorov GeorgiSTodorov merged commit 9d0ddd3 into master Jun 21, 2019
@GeorgiSTodorov GeorgiSTodorov deleted the brandonc/lingui_2.8.0 branch June 21, 2019 07:13
@mesosphere-ci
Copy link
Collaborator

🎉 This PR is included in version 2.111.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Pressure Review merge when having some slack released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants