Skip to content

Conversation

jzolnowski
Copy link

@jzolnowski jzolnowski commented Aug 7, 2023

After executing npm install, package-lock.json was also updated to reflect the changes from package.json.
In the process of updating the lockfile, several versions of the dependencies were upgraded automatically.

@jzolnowski jzolnowski requested review from Splaktar and gkalpak August 7, 2023 06:56
Copy link

@gkalpak gkalpak left a comment

Choose a reason for hiding this comment

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

We should also update the lockfile.

@jzolnowski jzolnowski force-pushed the chore-bump-angularjs branch from aeb1fa4 to eb81c89 Compare August 10, 2023 15:03
@jzolnowski jzolnowski requested a review from gkalpak August 10, 2023 15:04
Copy link

@gkalpak gkalpak left a comment

Choose a reason for hiding this comment

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

This PR now contains more updates than just "angular.js to v1.9.2".
We should either limit the changes to AngularJS or update the commit message to reflect the actual changes.

@jzolnowski jzolnowski force-pushed the chore-bump-angularjs branch from eb81c89 to 86b7dad Compare August 11, 2023 14:04
@jzolnowski jzolnowski requested a review from gkalpak August 11, 2023 14:04
@jzolnowski jzolnowski changed the title chore(package.json): bump angular.js to v1.9.2 chore(package.json): bump angular.js and angular.js-based libraries to v1.9.2 Aug 11, 2023
Copy link

@gkalpak gkalpak left a comment

Choose a reason for hiding this comment

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

The changes lgtm, but I find the commit message confusing/unclear. What does "angular.js-based libraries" refer to?

@jzolnowski
Copy link
Author

The changes lgtm, but I find the commit message confusing/unclear. What does "angular.js-based libraries" refer to?

angular.js-based librarie means all the rest bumped angularjs-based packages like:

"angular-animate": "npm:@xlts.dev/angular-animate@^1.9.2",
"angular-aria": "npm:@xlts.dev/angular-aria@^1.9.2",
"angular-material": "npm:@xlts.dev/angular-material@^1.2.4",
"angular-messages": "npm:@xlts.dev/angular-messages@^1.9.2"

@jzolnowski jzolnowski requested a review from gkalpak August 14, 2023 13:02
Copy link

@gkalpak gkalpak left a comment

Choose a reason for hiding this comment

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

The changes lgtm, but I find the commit message confusing/unclear. What does "angular.js-based libraries" refer to?

angular.js-based librarie means all the rest bumped angularjs-based packages like:

"angular-animate": "npm:@xlts.dev/angular-animate@^1.9.2",
"angular-aria": "npm:@xlts.dev/angular-aria@^1.9.2",
"angular-material": "npm:@xlts.dev/angular-material@^1.2.4",
"angular-messages": "npm:@xlts.dev/angular-messages@^1.9.2"

I see. I don't think we need to explicitly mention these libraries in the commit message, since they are part of the AngularJS framework and can be collectively referred to as AngularJS.

But this PR also updates other non-AngularJS related dependencies. We should either mention that in the commit message to move those other updates to a separate commit/PR.

@jzolnowski jzolnowski changed the title chore(package.json): bump angular.js and angular.js-based libraries to v1.9.2 chore(package.json): update angular.js to v1.9.2 along with other non-angularjs related dependencies Aug 16, 2023
@jzolnowski jzolnowski force-pushed the chore-bump-angularjs branch from 86b7dad to 1ba8516 Compare August 16, 2023 16:04
@jzolnowski jzolnowski requested a review from gkalpak August 18, 2023 09:57
Copy link

@gkalpak gkalpak left a comment

Choose a reason for hiding this comment

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

We should now update to v1.9.3 😉

…-angularjs related dependencies

After executing `npm install`, `package-lock.json` was also updated to reflect the changes from `package.json`.
In the process of updating the lockfile, several versions of the dependencies were upgraded automatically.
@jzolnowski jzolnowski force-pushed the chore-bump-angularjs branch from 1ba8516 to 832ca7c Compare August 22, 2023 09:25
@jzolnowski jzolnowski changed the title chore(package.json): update angular.js to v1.9.2 along with other non-angularjs related dependencies chore(package.json): update angular.js to v1.9.3 along with other non-angularjs related dependencies Aug 22, 2023
@jzolnowski jzolnowski requested a review from gkalpak August 22, 2023 09:25
@jzolnowski
Copy link
Author

We should now update to v1.9.3 😉

Done

@gkalpak gkalpak merged commit 0f8769a into master Aug 22, 2023
@gkalpak gkalpak deleted the chore-bump-angularjs branch August 22, 2023 18:01
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.

2 participants