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

Fixing translations job in this repo #321

Merged
merged 3 commits into from
Jul 13, 2022

Conversation

matthugs
Copy link
Contributor

There was a peer dependency issue introduced in new versions of upstream JS libraries which caused these automatic updates to the translations of this repo to fail. Alie's contributed a version bump that avoids that dependency issue, and my commit syncs the lockfile with that version bump so we can hopefully get clean ci.

@codecov
Copy link

codecov bot commented Jul 12, 2022

Codecov Report

Merging #321 (9c76d96) into master (1bd884c) will increase coverage by 0.50%.
The diff coverage is 97.95%.

@@            Coverage Diff             @@
##           master     #321      +/-   ##
==========================================
+ Coverage   73.38%   73.89%   +0.50%     
==========================================
  Files         102      105       +3     
  Lines        1924     1965      +41     
  Branches      455      475      +20     
==========================================
+ Hits         1412     1452      +40     
  Misses        485      485              
- Partials       27       28       +1     
Impacted Files Coverage Δ
src/pages-and-resources/live/messages.js 100.00% <ø> (ø)
src/pages-and-resources/live/data/api.js 92.68% <50.00%> (-2.19%) ⬇️
src/pages-and-resources/live/BBBSettings.jsx 100.00% <100.00%> (ø)
src/pages-and-resources/live/LiveCommonFields.jsx 100.00% <100.00%> (ø)
src/pages-and-resources/live/Settings.jsx 90.00% <100.00%> (+0.25%) ⬆️
src/pages-and-resources/live/ZoomSettings.jsx 100.00% <100.00%> (ø)
src/pages-and-resources/live/constants.js 100.00% <100.00%> (ø)
...-and-resources/live/factories/mockApiResponses.jsx 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d6997b8...9c76d96. Read the comment docs.

Copy link
Contributor

@connorhaugh connorhaugh left a comment

Choose a reason for hiding this comment

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

LGTM TYSM

@connorhaugh connorhaugh merged commit 21c36a4 into master Jul 13, 2022
@connorhaugh connorhaugh deleted the transifex-bot-update-translations2022-07-10 branch July 13, 2022 12:35
bradenmacdonald pushed a commit to open-craft/frontend-app-authoring that referenced this pull request Aug 9, 2024
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.

4 participants