Skip to content

fix: fix typings for queries with {plain: true} option#13899

Merged
ephys merged 3 commits into
sequelize:mainfrom
arsonik:plain-typings
Jan 5, 2022
Merged

fix: fix typings for queries with {plain: true} option#13899
ephys merged 3 commits into
sequelize:mainfrom
arsonik:plain-typings

Conversation

@arsonik
Copy link
Copy Markdown
Contributor

@arsonik arsonik commented Jan 4, 2022

Description Of Change

Add null to returned Promise when querying with {plain: true}

@ephys ephys added the typescript For issues and PRs. Things that involve typescript, such as typings and intellisense. label Jan 4, 2022
@arsonik
Copy link
Copy Markdown
Contributor Author

arsonik commented Jan 4, 2022

I guess it should wait for #13870 to be merged

@WikiRik
Copy link
Copy Markdown
Member

WikiRik commented Jan 4, 2022

I guess it should wait for #13870 to be merged

Don't worry about that. The tests are re-running, but we can approve this PR without disabling the flaky test

@arsonik
Copy link
Copy Markdown
Contributor Author

arsonik commented Jan 4, 2022

I guess it should wait for #13870 to be merged

Don't worry about that. The tests are re-running, but we can approve this PR without disabling the flaky test

ok its all good then ?

@ephys
Copy link
Copy Markdown
Member

ephys commented Jan 5, 2022

Tested and looks good to me. Now it's just a question of merging :)

@ephys ephys changed the title Fix typings for queries with {plain: true} option fix: fix typings for queries with {plain: true} option Jan 5, 2022
@ephys ephys merged commit d139b2e into sequelize:main Jan 5, 2022
sdepold pushed a commit that referenced this pull request Jan 10, 2022
Co-authored-by: Rik Smale <13023439+WikiRik@users.noreply.github.com>
Co-authored-by: fncolon <fncolon@pm.me>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 7, 2022

🎉 This PR is included in version 7.0.0-alpha.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

aliatsis pushed a commit to creditiq/sequelize that referenced this pull request Jun 2, 2022
Co-authored-by: Rik Smale <13023439+WikiRik@users.noreply.github.com>
Co-authored-by: fncolon <fncolon@pm.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released on @v7 typescript For issues and PRs. Things that involve typescript, such as typings and intellisense.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants