Skip to content

fix: Update pull_translations to include additional sources and imports#8

Open
rpipaliya-sonata wants to merge 1 commit into
masterfrom
fix/COSMO2-810
Open

fix: Update pull_translations to include additional sources and imports#8
rpipaliya-sonata wants to merge 1 commit into
masterfrom
fix/COSMO2-810

Conversation

@rpipaliya-sonata
Copy link
Copy Markdown

This pull request updates the translation pulling process in the Makefile to improve flexibility and support for additional sources. The main changes involve parameterizing extra translation sources and imports, and updating the list of supported frontend apps.

Translation process improvements:

  • Added new variables (ATLAS_EXTRA_SOURCES, ATLAS_EXTRA_INTL_IMPORTS, ATLAS_OPTIONS) to allow specifying additional translation sources and imports externally, such as from edx-internal.
  • Updated the list of translation sources and imports to include frontend-app-program-console instead of frontend-app-program-manager, and to append any extra sources/imports provided via the new variables.

JIRA

Copilot AI review requested due to automatic review settings May 14, 2026 06:14
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Makefile translation pull workflow to better support Atlas-based translation imports for this frontend app and allow downstream customization.

Changes:

  • Adds configurable Atlas variables for extra sources, extra intl imports, and Atlas options.
  • Updates the Atlas source/import target from program-manager to program-console.
  • Appends externally supplied translation sources/imports to the pull workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coveralls
Copy link
Copy Markdown

coveralls commented May 14, 2026

Coverage Report for CI Build 25845090840

Coverage remained the same at 55.335%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 306
Covered Lines: 168
Line Coverage: 54.9%
Relevant Branches: 97
Covered Branches: 55
Branch Coverage: 56.7%
Branches in Coverage %: Yes
Coverage Strength: 6.01 hits per line

💛 - Coveralls

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.

3 participants