Conversation
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* fix: Run `npm audit fix` to update dependencies. * chore(deps): update dependency @openedx/paragon to v23.14.9 (openedx#735) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): update dependency @reduxjs/toolkit to v2.9.1 (openedx#736) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): bump actions/setup-node from 5 to 6 (openedx#737) --------- Co-authored-by: Feanil Patel <feanil@axim.org> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…9.3 (openedx#739) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…edx#740) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…enedx#744) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…extra repositories (openedx#752)
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…se-ulmo (#8) * fix: Run `npm audit fix` to update dependencies. * chore(deps): update dependency @openedx/paragon to v23.14.9 (openedx#735) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): update dependency @reduxjs/toolkit to v2.9.1 (openedx#736) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): bump actions/setup-node from 5 to 6 (openedx#737) * chore(deps): update dependency @edx/frontend-component-footer to v14.9.3 (openedx#739) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): update dependency @edx/frontend-platform to v8.5.2 (openedx#740) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): update dependency @reduxjs/toolkit to v2.9.2 (openedx#741) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): update dependency @openedx/paragon to v23.16.0 (openedx#742) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): update dependency @reduxjs/toolkit to v2.10.1 (openedx#743) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * fix(deps): update dependency @edx/frontend-component-header to v8 (openedx#744) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * [DEPR] feat!: remove notices wrapper (openedx#731) * fix(deps): update dependency react-router-dom to v6.30.2 (openedx#745) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): update dependency @openedx/paragon to v23.17.0 (openedx#746) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): update dependency @openedx/paragon to v23.18.0 (openedx#748) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): update dependency @reduxjs/toolkit to v2.11.0 (openedx#749) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * feat: added the ability for instances to use local translations from extra repositories (openedx#752) * feat: added the ability for instances to use local translations from extra repositories (openedx#752) --------- Co-authored-by: Feanil Patel <feanil@axim.org> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ejaz Ahmad <86868918+jajjibhai008@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Maxwell Frank <92897870+MaxFrank13@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Synces the release-ulmo branch with the latest openedx/master, primarily by aligning dependencies, build tooling, and translation/CI configuration with upstream.
Changes:
- Update frontend dependencies (e.g.,
core-js,react-router-dom) and regeneratepackage-lock.jsonaccordingly. - Extend translation pulling/importing to include
frontend-component-header. - Update GitHub Actions CI workflow (notably
actions/checkoutversion).
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | Bumps/selectively removes dependency entries to match upstream. |
| package-lock.json | Regenerated lockfile reflecting updated transitive dependency graph. |
| Makefile | Adds frontend-component-header to Atlas translation pull + intl imports. |
| .github/workflows/ci.yml | Adjusts CI checkout action version (workflow structure impacted). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v5 | ||
| uses: actions/checkout@v6 | ||
|
|
There was a problem hiding this comment.
In this workflow, the list items under steps: are indented at the same level as the steps: key. YAML requires the sequence to be indented under the key (e.g., steps: then - name: ...), otherwise the workflow file can fail to parse and CI will not run. Please adjust the indentation so the - name: entries are nested under steps:.
jsnwesson
left a comment
There was a problem hiding this comment.
I think pretty soon we'll need to move our branch from release-ulmo to master, but this looks good
|
@MaxFrank13 well... wait. Maybe we shouldn't do this. I think we should keep the |
I think this PR should update the master branch in this forked repo, instead of release-ulmo so that if we deploy off of master and it breaks, we can at least revert back to using release-ulmo
Sync the
edx/masterbranch with the latest fromopenedx/masterhttps://2u-internal.atlassian.net/browse/SUBS-23