fix(deps): update dependency slonik to v37.4.0 #1845
Conversation
![renovate[bot]](https://avatars.githubusercontent.com/in/2740?s=60&v=4){kind=small}
|
Important Auto Review SkippedBot user detected. To trigger a single review, invoke the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
|
Here's the code health analysis summary for commits Analysis Summary
|
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/slonik@37.3.1 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found Additional details and impacted files@@ Coverage Diff @@
## main #1845 +/- ##
=======================================
Coverage 35.62% 35.62%
=======================================
Files 127 127
Lines 12063 12063
Branches 219 137 -82
=======================================
Hits 4298 4298
Misses 7765 7765
☔ View full report in Codecov by Sentry. |
renovate
bot
force-pushed
the
renovate/slonik-37.x-lockfile
branch
from
05ee70c
to
1fb1a11
Compare
![codecov[bot]](https://avatars.githubusercontent.com/in/254?s=60&v=4){kind=small}
There was a problem hiding this comment.
The diff indicates version upgrades for the 'slonik' and '@types/pg' packages in pnpm-lock.yaml file. Ensure these version upgrades do not bring breaking changes to your project. Always have tests in place to catch unexpected issues that may arise due to version changes.
| @@ -37,7 +37,7 @@ importers: | |||
| version: 5.0.3 | |||
| slonik: | |||
| specifier: '37' | |||
| version: 37.3.1(zod@3.22.4) | |||
| version: 37.4.0(zod@3.22.4) | |||
There was a problem hiding this comment.
Here 'slonik' package has been upgraded from version 37.3.1 to 37.4.0. Make sure the changes in this version do not break the current functionality of the project.
| @@ -53,7 +53,7 @@ importers: | |||
| version: 7.108.0 | |||
| '@slonik/migrator': | |||
| specifier: 0.12.0 | |||
| version: 0.12.0(@types/node@20.11.30)(slonik@37.3.1) | |||
There was a problem hiding this comment.
Here '@slonik/migrator' package also reflects the version change of 'slonik' from 37.3.1 to 37.4.0. Please ensure that this does not conflict with '@slonik/migrator' functionality.
| @@ -104,7 +104,7 @@ importers: | |||
| version: 7.108.0 | |||
| slonik: | |||
| specifier: ^37.2.0 | |||
There was a problem hiding this comment.
Another 'slonik' package upgrade instance. Verify that all instances of 'slonik' package in your project are updated to maintain consistency.
| @@ -1245,13 +1245,13 @@ packages: | |||
| resolution: {integrity: sha512-+Fj43pSMwJs4KRrH/938Uf+uAELIgVBmQzg/q1YG10djyfA3TnrU8N8XzqCh/okZdszqBQTZf96idMfE5lnwTA==} | |||
| dev: true | |||
|
|
|||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.3.1): | |||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.4.0): | |||
| resolution: {integrity: sha512-2kAXCO/iW9/VOYlfl8Jza4fQpi5Q8+JUyB38YFUxPiWNc+6o41GGND2aIpnyr6EPFNRXdg5x+Fu8+SMyMvMXpQ==} | |||
| peerDependencies: | |||
| slonik: 27 - 29 | |||
| dependencies: | |||
There was a problem hiding this comment.
The 'slonik' package update is reflected here as well. Make sure this doesn't affect the dependencies and transitive dependencies linked with it.
| @@ -1274,8 +1274,8 @@ packages: | |||
| dependencies: | |||
| undici-types: 5.26.5 | |||
There was a problem hiding this comment.
Here '@types/pg' package has been upgraded from version 8.11.2 to 8.11.4. Check if this version change introduces new features or bug fixes which may be beneficial or disruptive to your project.
| @@ -3824,13 +3824,13 @@ packages: | |||
| engines: {node: '>=14'} | |||
| dev: true | |||
There was a problem hiding this comment.
Again, the 'slonik' package has been updated here which reaffirms the need to ensure the update's compatibility with various parts of the project.
| /slonik@37.3.1(zod@3.22.4): | ||
| resolution: {integrity: sha512-vidC0OleHqIrFt5gGWgwy4CUhE+dSo71/661g2MyQFNpT/ZyiBZ1BscGK8gLlw4K8xYTo4SL+POBBLp+X5wYsg==} | ||
| /slonik@37.4.0(zod@3.22.4): | ||
| resolution: {integrity: sha512-W9GPnc4M5+JLvrl4aMx2qkpIQrHLo6vAmCc2UeTD5Szl2uBA2ChZo3njdygK1511wrlSSDnOySY+jlLNAfpIvA==} | ||
| engines: {node: '>=18'} |
There was a problem hiding this comment.
The '@types/pg' version update is seen here again. Ensure that all package instances are successfully updated.
There was a problem hiding this comment.
The changes in this Git diff primarily update versions for multiple packages including 'slonik' and '@types/pg'. However, a potential issue is found with the peerDependencies of '@slonik/migrator'. It specifies its 'slonik' peer dependency taking versions 27 to 29 while the slonik package itself is being updated to version 37.4.0 which can cause version conflict problems.
| @@ -37,7 +37,7 @@ importers: | |||
| version: 5.0.3 | |||
| slonik: | |||
| specifier: '37' | |||
| version: 37.3.1(zod@3.22.4) | |||
| version: 37.4.0(zod@3.22.4) | |||
There was a problem hiding this comment.
The 'slonik' package is being updated from version 37.3.1 to 37.4.0. Make sure all usages are tested and any breaking changes in the new version are accounted for.
| @@ -53,7 +53,7 @@ importers: | |||
| version: 7.108.0 | |||
| '@slonik/migrator': | |||
| specifier: 0.12.0 | |||
| version: 0.12.0(@types/node@20.11.30)(slonik@37.3.1) | |||
There was a problem hiding this comment.
@slonik/migrator is depending on 'slonik' version 37.4.0. The dependent libraries need to be thoroughly tested with this updated version.
| @@ -1245,13 +1245,13 @@ packages: | |||
| resolution: {integrity: sha512-+Fj43pSMwJs4KRrH/938Uf+uAELIgVBmQzg/q1YG10djyfA3TnrU8N8XzqCh/okZdszqBQTZf96idMfE5lnwTA==} | |||
| dev: true | |||
|
|
|||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.3.1): | |||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.4.0): | |||
There was a problem hiding this comment.
This line defines the peerDependency for '@slonik/migrator' to accept 'slonik' versions from 27 to 29. This range doesn't include the new version of 'slonik' (37.4.0) being used, which could potentially lead to dependency errors.
| @@ -1245,13 +1245,13 @@ packages: | |||
| resolution: {integrity: sha512-+Fj43pSMwJs4KRrH/938Uf+uAELIgVBmQzg/q1YG10djyfA3TnrU8N8XzqCh/okZdszqBQTZf96idMfE5lnwTA==} | |||
| dev: true | |||
|
|
|||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.3.1): | |||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.4.0): | |||
| resolution: {integrity: sha512-2kAXCO/iW9/VOYlfl8Jza4fQpi5Q8+JUyB38YFUxPiWNc+6o41GGND2aIpnyr6EPFNRXdg5x+Fu8+SMyMvMXpQ==} | |||
| peerDependencies: | |||
| slonik: 27 - 29 | |||
| dependencies: | |||
There was a problem hiding this comment.
This line shows '@slonik/migrator' directly depending on 'slonik' version 37.4.0, which doesn't align with its defined peerDependencies versions of 27 to 29.
| @@ -3824,13 +3824,13 @@ packages: | |||
| engines: {node: '>=14'} | |||
There was a problem hiding this comment.
The 'slonik' package is being updated from version 37.3.1 to 37.4.0. As this package is widely used across the project (as per lines 10, 28, 44), ensure all components depending on it are tested.
| @@ -1274,8 +1274,8 @@ packages: | |||
| dependencies: | |||
| undici-types: 5.26.5 | |||
There was a problem hiding this comment.
The '@types/pg' package is being updated from version 8.11.2 to 8.11.4. Ensure all areas of code leveraging this package are tested with this new version.
There was a problem hiding this comment.
The diff is mainly about version upgrades of different software dependencies. Slonik has been upgraded from version 37.3.1 to 37.4.0 in multiple parts of the pnpm-lock.yaml file. '@types/pg' package has been upgraded from 8.11.2 to 8.11.4. However, there seems to be inconsistency about the peer dependencies for 'slonik' which could be an error.
| @@ -37,7 +37,7 @@ importers: | |||
| version: 5.0.3 | |||
| slonik: | |||
| specifier: '37' | |||
| version: 37.3.1(zod@3.22.4) | |||
| version: 37.4.0(zod@3.22.4) | |||
There was a problem hiding this comment.
The version of 'slonik' has been upgraded to 37.4.0 from 37.3.1. Make sure the higher version doesn't break any functionalities.
| @@ -53,7 +53,7 @@ importers: | |||
| version: 7.108.0 | |||
| '@slonik/migrator': | |||
| specifier: 0.12.0 | |||
| version: 0.12.0(@types/node@20.11.30)(slonik@37.3.1) | |||
There was a problem hiding this comment.
The version of 'slonik' for '@slonik/migrator' has been upgraded to 37.4.0 from 37.3.1. Check whether the new version of 'slonik' is compatible with '@slonik/migrator'.
| @@ -104,7 +104,7 @@ importers: | |||
| version: 7.108.0 | |||
| slonik: | |||
| specifier: ^37.2.0 | |||
There was a problem hiding this comment.
The primary version of the 'slonik' has also been upgraded to 37.4.0 from 37.3.1. All instances of 'slonik' are now referring to 37.4.0.
| @@ -1245,13 +1245,13 @@ packages: | |||
| resolution: {integrity: sha512-+Fj43pSMwJs4KRrH/938Uf+uAELIgVBmQzg/q1YG10djyfA3TnrU8N8XzqCh/okZdszqBQTZf96idMfE5lnwTA==} | |||
| dev: true | |||
There was a problem hiding this comment.
Version updates for '@slonik/migrator' reflecting the upgraded version of 'slonik'. Ensure the changes do not introduce dependencies issues.
| @@ -1245,13 +1245,13 @@ packages: | |||
| resolution: {integrity: sha512-+Fj43pSMwJs4KRrH/938Uf+uAELIgVBmQzg/q1YG10djyfA3TnrU8N8XzqCh/okZdszqBQTZf96idMfE5lnwTA==} | |||
| dev: true | |||
|
|
|||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.3.1): | |||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.4.0): | |||
| resolution: {integrity: sha512-2kAXCO/iW9/VOYlfl8Jza4fQpi5Q8+JUyB38YFUxPiWNc+6o41GGND2aIpnyr6EPFNRXdg5x+Fu8+SMyMvMXpQ==} | |||
| peerDependencies: | |||
| slonik: 27 - 29 | |||
| dependencies: | |||
There was a problem hiding this comment.
Update in the dependencies for '@slonik/migrator', 'slonik' appears to be updated here as well.
| @@ -1245,13 +1245,13 @@ packages: | |||
| resolution: {integrity: sha512-+Fj43pSMwJs4KRrH/938Uf+uAELIgVBmQzg/q1YG10djyfA3TnrU8N8XzqCh/okZdszqBQTZf96idMfE5lnwTA==} | |||
| dev: true | |||
|
|
|||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.3.1): | |||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.4.0): | |||
There was a problem hiding this comment.
The peerDependencies for 'slonik' seems to be mentioning version 27-29, but the actual use of 'slonik' is of version 37.4.0. This inconsistency should be corrected.
| @@ -1274,8 +1274,8 @@ packages: | |||
| dependencies: | |||
| undici-types: 5.26.5 | |||
|
|
|||
There was a problem hiding this comment.
'@types/pg' version is updated to 8.11.4 from 8.11.2. Similar to the other updates, please ensure that this upgrade does not cause any issues.
| @@ -3824,13 +3824,13 @@ packages: | |||
| engines: {node: '>=14'} | |||
| dev: true | |||
There was a problem hiding this comment.
'slonik' version update is reflected in the resolution as well. Please cross-check the integrity hash to ensure high security.
| /slonik@37.3.1(zod@3.22.4): | ||
| resolution: {integrity: sha512-vidC0OleHqIrFt5gGWgwy4CUhE+dSo71/661g2MyQFNpT/ZyiBZ1BscGK8gLlw4K8xYTo4SL+POBBLp+X5wYsg==} | ||
| /slonik@37.4.0(zod@3.22.4): | ||
| resolution: {integrity: sha512-W9GPnc4M5+JLvrl4aMx2qkpIQrHLo6vAmCc2UeTD5Szl2uBA2ChZo3njdygK1511wrlSSDnOySY+jlLNAfpIvA==} | ||
| engines: {node: '>=18'} |
There was a problem hiding this comment.
The 'slonik' upgrade seems to affect the dependency to '@types/pg' as well, upgrading it to 8.11.4 from 8.11.2. Make sure new version of '@types/pg' works correctly with 'slonik'.
There was a problem hiding this comment.
The changes are primarily updating the versions of packages namely '@slonik/migrator', 'slonik', and '@types/pg'. Although these upgrades are good for getting the latest features and bug fixes, we should ensure that the updated versions don't introduce any breaking changes or have any known issues that could affect our application.
| @@ -37,7 +37,7 @@ importers: | |||
| version: 5.0.3 | |||
| slonik: | |||
| specifier: '37' | |||
| version: 37.3.1(zod@3.22.4) | |||
| version: 37.4.0(zod@3.22.4) | |||
There was a problem hiding this comment.
The version of the slonik package has been upgraded from 37.3.1 to 37.4.0. This is generally fine, but it would be necessary to ensure that this new version works correctly with any code that depends on this package. It's highly recommended to have unit tests that verify said behavior.
| @@ -53,7 +53,7 @@ importers: | |||
| version: 7.108.0 | |||
| '@slonik/migrator': | |||
| specifier: 0.12.0 | |||
| version: 0.12.0(@types/node@20.11.30)(slonik@37.3.1) | |||
There was a problem hiding this comment.
'@slonik/migrator' version is updated to be used with 'slonik' version 37.4.0. Please make sure all dependencies of '@slonik/migrator' are compatible with updated 'slonik' version.
| @@ -104,7 +104,7 @@ importers: | |||
| version: 7.108.0 | |||
| slonik: | |||
| specifier: ^37.2.0 | |||
There was a problem hiding this comment.
The version of 'slonik' has again been updated from 37.3.1 to 37.4.0 in another section of the code. Ensure the consistency of package version across your whole project.
| @@ -1245,13 +1245,13 @@ packages: | |||
| resolution: {integrity: sha512-+Fj43pSMwJs4KRrH/938Uf+uAELIgVBmQzg/q1YG10djyfA3TnrU8N8XzqCh/okZdszqBQTZf96idMfE5lnwTA==} | |||
| dev: true | |||
There was a problem hiding this comment.
'@slonik/migrator' version is changed along with 'slonik' version change. It’s good that dependencies are kept in sync but detailed testing is necessary before deployment.
| @@ -1245,13 +1245,13 @@ packages: | |||
| resolution: {integrity: sha512-+Fj43pSMwJs4KRrH/938Uf+uAELIgVBmQzg/q1YG10djyfA3TnrU8N8XzqCh/okZdszqBQTZf96idMfE5lnwTA==} | |||
| dev: true | |||
|
|
|||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.3.1): | |||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.4.0): | |||
| resolution: {integrity: sha512-2kAXCO/iW9/VOYlfl8Jza4fQpi5Q8+JUyB38YFUxPiWNc+6o41GGND2aIpnyr6EPFNRXdg5x+Fu8+SMyMvMXpQ==} | |||
| peerDependencies: | |||
| slonik: 27 - 29 | |||
| dependencies: | |||
There was a problem hiding this comment.
Change in 'slonik' version. It’s always better to keep all instances of packages updated to the same version to avoid inconsistencies.
| @@ -1274,8 +1274,8 @@ packages: | |||
| dependencies: | |||
| undici-types: 5.26.5 | |||
There was a problem hiding this comment.
'@types/pg' version is updated from 8.11.2 to 8.11.4. It's important to check the changelogs for any potential breaking changes.
| @@ -3824,13 +3824,13 @@ packages: | |||
| engines: {node: '>=14'} | |||
There was a problem hiding this comment.
The 'slonik' package version upgrade is reflected here again. This version change should be cross-checked across all components to ensure that no breaking changes are introduced.
| /slonik@37.3.1(zod@3.22.4): | ||
| resolution: {integrity: sha512-vidC0OleHqIrFt5gGWgwy4CUhE+dSo71/661g2MyQFNpT/ZyiBZ1BscGK8gLlw4K8xYTo4SL+POBBLp+X5wYsg==} | ||
| /slonik@37.4.0(zod@3.22.4): | ||
| resolution: {integrity: sha512-W9GPnc4M5+JLvrl4aMx2qkpIQrHLo6vAmCc2UeTD5Szl2uBA2ChZo3njdygK1511wrlSSDnOySY+jlLNAfpIvA==} | ||
| engines: {node: '>=18'} |
There was a problem hiding this comment.
Updating '@types/pg' to 8.11.4. This should bring it in line with the other updates, but again, make sure your code is compatible with these new versions.
renovate
bot
force-pushed
the
renovate/slonik-37.x-lockfile
branch
from
1fb1a11
to
274ed91
Compare
Bundle ReportBundle size has no change ✅ |
| @@ -1287,8 +1287,8 @@ packages: | |||
| dependencies: | |||
| undici-types: 5.26.5 | |||
|
|
|||
There was a problem hiding this comment.
The version '@types/pg' has been updated here as well. Make sure the changes correlate with the all other changes and the updated version of 'pg' is compatible with '@types/pg'.
| @@ -3826,13 +3826,13 @@ packages: | |||
| engines: {node: '>=14'} | |||
| dev: true | |||
There was a problem hiding this comment.
The updated version of 'slonik' is installed here as well. Consistency is maintained.
| @@ -1258,13 +1258,13 @@ packages: | |||
| resolution: {integrity: sha512-+Fj43pSMwJs4KRrH/938Uf+uAELIgVBmQzg/q1YG10djyfA3TnrU8N8XzqCh/okZdszqBQTZf96idMfE5lnwTA==} | |||
| dev: true | |||
|
|
|||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.3.1): | |||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.4.0): | |||
There was a problem hiding this comment.
Please check the versions of 'slonik' in peerDependencies of '@slonik/migrator'. The versions are way behind the current and the upgraded versions.
| @@ -1287,8 +1287,8 @@ packages: | |||
| dependencies: | |||
| undici-types: 5.26.5 | |||
|
|
|||
There was a problem hiding this comment.
The version of the @types/pg package has been upgraded from 8.11.2 to 8.11.4. As with the previous comment, it's necessary to verify that the changes between these versions do not affect the way your software operates. A proper set of unit tests could thoroughly validate this.
| @@ -1258,13 +1258,13 @@ packages: | |||
| resolution: {integrity: sha512-+Fj43pSMwJs4KRrH/938Uf+uAELIgVBmQzg/q1YG10djyfA3TnrU8N8XzqCh/okZdszqBQTZf96idMfE5lnwTA==} | |||
| dev: true | |||
|
|
|||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.3.1): | |||
| /@slonik/migrator@0.12.0(@types/node@20.11.30)(slonik@37.4.0): | |||
There was a problem hiding this comment.
In this line, it specifies that the version of slonik is expected to be between 27 and 29 by the @slonik/migrator package. However, as seen in the recent changes, the slonik version has become 37.4.0, which doesn't satisfy this condition. This conflict can potentially cause issues at runtime and it would be better to clarify the need for the slonik version requirements for @slonik/migrator or to have the version updated to match the expectations.
This PR contains the following updates:
37.3.1->37.4.0Release Notes
gajus/slonik (slonik)
v37.4.0Compare Source
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled because a matching PR was automerged previously.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR has been generated by Mend Renovate. View repository job log here.