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.

Sign up for GitHub

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

Jump to bottom

Upgrade packages for 1.0.0 #60

Merged
merged 1 commit into from
Feb 28, 2024
Merged

Upgrade packages for 1.0.0 #60

merged 1 commit into from
Feb 28, 2024

Conversation

enov
Copy link
Collaborator

@enov enov commented Feb 28, 2024 

  merge-deep          ^3.0.2  →   ^3.0.3
  mocha              ^10.2.0  →  ^10.3.0
  router              ^1.3.3  →   ^1.3.8
  serve-static       ^1.13.2  →  ^1.15.0
  supertest           ^6.3.3  →   ^6.3.4
  swagger-ui-dist     ^5.4.2  →  ^5.11.8
  yaml                ^2.3.1  →   ^2.4.0

@enov
Upgrade dependencies
a142304 
 - merge-deep          ^3.0.2  →   ^3.0.3
 - mocha              ^10.2.0  →  ^10.3.0
 - router              ^1.3.3  →   ^1.3.8
 - serve-static       ^1.13.2  →  ^1.15.0
 - supertest           ^6.3.3  →   ^6.3.4
 - swagger-ui-dist     ^5.4.2  →  ^5.11.8
 - yaml                ^2.3.1  →   ^2.4.0
@enov enov requested a review from wesleytodd as a code owner February 28, 2024 00:19
@enov enov requested a review from Megapixel99 February 28, 2024 00:19
Megapixel99
Megapixel99 approved these changes Feb 28, 2024
View reviewed changes
Copy link
Collaborator

@Megapixel99 Megapixel99 left a comment

Choose a reason for hiding this comment

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

None of these seem to be breaking changes, looks okay to me.

@enov
Copy link
Collaborator Author

enov commented Feb 28, 2024

Probably not a necessary pull request, because npm i on a project using this package would install the latest non-breaking versions because of the use of ^ in the version specs in package.json.

enov
enov commented Feb 28, 2024
View reviewed changes
package.json Show resolved Hide resolved
@wesleytodd
Copy link
Owner

I think even when npm i would pick it up, since this is a 1.0.0 we should attempt to get them forcibly updated as possiable. So thanks for doing this! Will merge and publish asap.

@wesleytodd wesleytodd merged commit 219bd06 into release/v1.0.0 Feb 28, 2024
5 checks passed
@wesleytodd wesleytodd deleted the bugfix/upgrade-packages-for-1.0.0 branch February 28, 2024 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Megapixel99 Megapixel99 Megapixel99 approved these changes

@wesleytodd wesleytodd wesleytodd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@enov @wesleytodd @Megapixel99