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

Fix sequelize/types/lib/hooks path #1202

Merged
merged 3 commits into from
Feb 11, 2022

Conversation

mystylelee
Copy link
Contributor

@mystylelee mystylelee commented Feb 11, 2022

#1198 Problem solved.
'types/lib' folder was removed in sequelize verstion >= 6,

test/specs/instance.spec.ts file edit - Model.reload() method is return Model,

Deprecated: Operator Aliases

dependabot bot and others added 3 commits February 10, 2022 02:02
Bumps [sequelize](https://github.com/sequelize/sequelize) from 6.8.0 to 6.16.1.
- [Release notes](https://github.com/sequelize/sequelize/releases)
- [Commits](sequelize/sequelize@v6.8.0...v6.16.1)

---
updated-dependencies:
- dependency-name: sequelize
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
'types/lib' folder was removed in sequelize verstion >= 6
@mystylelee mystylelee changed the title Feature/#1198 Fix sequelize/types/lib/hooks path Feb 11, 2022
@codecov
Copy link

codecov bot commented Feb 11, 2022

Codecov Report

Merging #1202 (61507a8) into master (9f1d033) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1202   +/-   ##
=======================================
  Coverage   95.11%   95.11%           
=======================================
  Files         117      117           
  Lines        1126     1126           
  Branches      129      129           
=======================================
  Hits         1071     1071           
  Misses         23       23           
  Partials       32       32           
Impacted Files Coverage Δ
src/hooks/shared/hooks-service.ts 88.57% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9f1d033...61507a8. Read the comment docs.

@lukashroch lukashroch merged commit ab45c14 into sequelize:master Feb 11, 2022
@mystylelee mystylelee deleted the feature/#1198 branch February 15, 2022 01:05
@stanleynguyen
Copy link

@lukashroch do we have a target date for when this will be released?

@lukashroch
Copy link
Collaborator

@stanleynguyen I've opened a PR to push out a release. So hopefully soon.

hsluoyz pushed a commit to node-casbin/sequelize-adapter that referenced this pull request Apr 28, 2022
2.1.3 is not compatible with sequelize 6.10.0 due to this change sequelize/sequelize-typescript#1202
samstefan added a commit to samstefan/sequelize-adapter that referenced this pull request Apr 29, 2022
This is to fix the security vulnerability CVE-2021-3765 in validator.js < 13.7.0

GHSA-qgmg-gppg-76g5

I've also fixed sequelize-typescript at version 2.1.2 as 2.1.3 is not compatible with sequelize 6.10.0 due to this change sequelize/sequelize-typescript#1202
hsluoyz pushed a commit to node-casbin/sequelize-adapter that referenced this pull request Apr 29, 2022
This is to fix the security vulnerability CVE-2021-3765 in validator.js < 13.7.0

GHSA-qgmg-gppg-76g5

I've also fixed sequelize-typescript at version 2.1.2 as 2.1.3 is not compatible with sequelize 6.10.0 due to this change sequelize/sequelize-typescript#1202
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.

4 participants