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.

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

chore: update dependencies and workflow, format package.json #25

Closed
wants to merge 2 commits into from

Conversation

YOUWILLDIE666
Copy link

Dependencies:
Bump rimraf to ^6.0.0
Bump ncc to 0.38.1
Bump pnpm to 9.10.0
Bump typescript to 5.6.2
Bump @types/node to v18.19.50
Bump Rojo to 7.4.4
Bump StyLua to 0.20.0

Workflow:
Jobs and Steps are now explicitly named
Bump upload-artifact, setup-node, checkout, and pnpm/action-setup actions to v4

Other:
Format package.json according to its intended scheme (ref. https://json.schemastore.org/package.json)

Dependencies:
Bump rimraf to ^6.0.0
Bump ncc to 0.38.1
Bump pnpm to 9.10.0
Bump typescript to 5.6.2
Bump @types/node to v18.19.50
Bump Rojo to 7.4.4
Bump StyLua to 0.20.0

Workflow:
Jobs and Steps are now explicitly named
Bump upload-artifact, setup-node, checkout, and pnpm/action-setup actions to v4

Other:
Format package.json according to its intended scheme (ref. https://json.schemastore.org/package.json)
@iArxic
Copy link
Owner

iArxic commented Sep 17, 2024

hey there, what is the point of updating the dependencies? I would merge this however it would not make a difference to the product itself

Since beginning with v6.0.0, rimraf is incompatible with Node.js versions prior to 20
@YOUWILLDIE666
Copy link
Author

YOUWILLDIE666 commented Sep 17, 2024

Hello, updating dependencies may resolve some bugs, reduce memory usage (possibly), etc.
I mean the app functioned well without the bumping too, making sure everything's up to date

EDIT: Also some of them are getting deprecated (e.g. rimraf versions prior to v4).
rimraf_deprecation

@YOUWILLDIE666 YOUWILLDIE666 closed this by deleting the head repository Oct 9, 2024
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