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

Revert "chore(deps): bump sqlite3 from 5.0.4 to 5.1.5" #1282

Merged
merged 1 commit into from
May 9, 2023

Conversation

ysfscream
Copy link
Member

Reverts #1247

@@ -48,7 +48,7 @@
"mqtt": "4.3.7",
"reflect-metadata": "^0.1.13",
"rxjs": "6.2.0",
"sqlite3": "^5.1.5",
"sqlite3": "^5.0.4",
"typedi": "^0.8.0",
"typeorm": "^0.2.34",
"typeorm-typedi-extensions": "^0.2.3",
Copy link

Choose a reason for hiding this comment

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

with the code review:

  1. The patch changes the version of the sqlite3 module from 5.1.5 to 5.0.4. It is important to make sure that the new version is compatible with the current project and that there are no breaking changes.

  2. We also need to check if there are any security vulnerabilities associated with this version change.

  3. Additionally, we should check if the new version is compatible with any other dependencies in the project, such as typeorm or rxjs.

  4. Finally, we need to run unit tests to ensure that the code change does not break anything.

mkdirp "^1.0.3"
yallist "^4.0.0"

tar@^6.1.11, tar@^6.1.2:
version "6.1.11"
resolved "https://registry.yarnpkg.com/tar/-/tar-6.1.11.tgz#6760a38f003afa1b2ffd0ffe9e9abbd0eab3d621"
integrity sha512-an/KZQzQUkZCkuoAA64hM92X0Urb6VpRhAFllDzz44U2mcD5scmT3zBc4VgVpkugF580+DQn8eAFSyoQt0tznA==
Copy link

Choose a reason for hiding this comment

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

with the code review.

The patch looks correct, as it updates the version of sqlite3 from 5.1.5 to 5.0.4 while also updating the dependencies of tar from 6.0.2 to 6.1.11 and 6.1.2.

It is important to make sure that the versions specified in the patch are compatible with the project's existing dependencies and that the new versions do not introduce any errors or bugs. Additionally, it is important to look for any security vulnerabilities associated with the updated versions and ensure that they are addressed.

@ysfscream ysfscream merged commit bf8f6cd into main May 9, 2023
@ysfscream ysfscream deleted the revert-1247-dependabot/npm_and_yarn/sqlite3-5.1.5 branch May 9, 2023 07:31
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.

1 participant