-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: major version updated for frontend-platform, paragon & frontend-build #20
feat: major version updated for frontend-platform, paragon & frontend-build #20
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #20 +/- ##
=======================================
Coverage 87.03% 87.03%
=======================================
Files 7 7
Lines 54 54
Branches 5 5
=======================================
Hits 47 47
Misses 7 7 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See https://discuss.openedx.org/t/npm-scope-change-for-open-edx-packages/11481 for more context.
"@edx/frontend-platform": "4.6.0", | ||
"@edx/paragon": "^20.44.0", | ||
"@edx/frontend-platform": "6.2.0", | ||
"@edx/paragon": "21.5.6", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Paragon has moved to the openedx
scope (@openedx/paragon
). Version 21.5.7
is the same content as 21.5.6
of @edx/paragon
To do this bump, you'll need to update to the new scope and update any imports of paragon to point to the new location.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There seems to be a PR already open for it, once merged I will rebase
@@ -45,7 +45,7 @@ | |||
}, | |||
"devDependencies": { | |||
"@edx/browserslist-config": "^1.1.1", | |||
"@edx/frontend-build": "12.8.38", | |||
"@edx/frontend-build": "13.0.14", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as with paragon
, frontend-build
has also moved to the openedx
scope (@openedx/frontend-build
), Version 13.0.15
there should be the same as this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There seems to be a PR already open for it, once merged I will rebase
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, I think this PR can be closed in favor of #21 since it updates all the libraries being updated here.
Description:
frontend-platform
,paragon
&frontend-build