Skip to content

Conversation

Doko-Demo-Doa
Copy link
Contributor

Updates the peer dependencies of query-mongoose and query-typegoose so that they can be installed with latest mongoose version.

@Doko-Demo-Doa Doko-Demo-Doa changed the title chore:update mongoose peerDependencies chore(deps):update mongoose peerDependencies Mar 29, 2024
Copy link

nx-cloud bot commented Mar 29, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit f95eabd. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


🟥 Failed Commands
nx run-many --target=lint --all
✅ Successfully ran 4 targets

Sent with 💌 from NxCloud.

@codecov-commenter
Copy link

codecov-commenter commented Mar 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.18%. Comparing base (c11da0c) to head (f95eabd).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #241      +/-   ##
==========================================
- Coverage   86.28%   79.18%   -7.10%     
==========================================
  Files         688      688              
  Lines        9761     9761              
  Branches      862      862              
==========================================
- Hits         8422     7729     -693     
- Misses        619     1284     +665     
- Partials      720      748      +28     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Doko-Demo-Doa
Copy link
Contributor Author

@TriPSs There are lots of @typescript-eslint/no-unnecessary-type-assertion linting issues, should I ignore them?

@TriPSs
Copy link
Owner

TriPSs commented Mar 29, 2024

Yea, need to fix the linting.

@TriPSs TriPSs merged commit e4b8286 into TriPSs:master Mar 29, 2024
@Doko-Demo-Doa
Copy link
Contributor Author

Hi @TriPSs please can we make a release on npm for this one?

@TriPSs
Copy link
Owner

TriPSs commented Mar 29, 2024

Yes ofc, sorry.

@Doko-Demo-Doa
Copy link
Contributor Author

Thank you. It works greatly now!

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.

3 participants