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(types): missing type for oracle dialect in v6 #14991

Closed
wants to merge 1 commit into from

Conversation

fzn0x
Copy link
Member

@fzn0x fzn0x commented Sep 16, 2022

The same as #14137

This is causing casting with any, just notice it when try the oracle DB.

Closes none but this is to prevent the future issues

@fzn0x fzn0x added the type: typescript For issues and PRs. Things that involve typescript, such as typings and intellisense. label Sep 16, 2022
@fzn0x fzn0x self-assigned this Sep 16, 2022
@fzn0x
Copy link
Member Author

fzn0x commented Sep 16, 2022

This changes targets v6 only since Oracle DB dialect was only in v6 branch.

@sdepold
Copy link
Member

sdepold commented Sep 16, 2022

👍

@sdepold
Copy link
Member

sdepold commented Sep 16, 2022

Please don't merge before answering this: Will the commit message cause a patch or a minor version bump? We should aim for the patch

@fzn0x
Copy link
Member Author

fzn0x commented Sep 16, 2022

Please don't merge before answering this: Will the commit message cause a patch or a minor version bump? We should aim for the patch

I'm agree with the patch option 👍

@sdepold
Copy link
Member

sdepold commented Sep 16, 2022

I think you need to change your commit message to fix instead of feat for that

@fzn0x fzn0x changed the title feat(types): missing type for oracle dialect in v6 fix(types): missing type for oracle dialect in v6 Sep 16, 2022
@fzn0x
Copy link
Member Author

fzn0x commented Sep 16, 2022

I think you need to change your commit message to fix instead of feat for that

Changed, is there anything required for the patch?

@sdepold
Copy link
Member

sdepold commented Sep 16, 2022

Can you rename the commit message too?

@fzn0x
Copy link
Member Author

fzn0x commented Sep 16, 2022

Can you rename the commit message too?

Looks like the branch does not allow rebase, maybe I will create another PR.

@fzn0x fzn0x closed this Sep 16, 2022
@fzn0x fzn0x deleted the v6-missing-dialect-types branch September 16, 2022 13:30
fzn0x added a commit that referenced this pull request Sep 16, 2022
WikiRik pushed a commit that referenced this pull request Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: 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.

None yet

2 participants