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

feat: backport ilike from next #6862

Merged
merged 1 commit into from
Oct 7, 2020
Merged

Conversation

imnotjames
Copy link
Contributor

the next branch has support for ilike and this backports
the FindOperator into the main branch

@imnotjames
Copy link
Contributor Author

ILike is not supported in oracle. Need to disable that test for oracle

@imnotjames
Copy link
Contributor Author

HMM, one of the mainline drivers don't support it either but I'm not sure which one. I'll have to check locally and then publish a change to the tests to disable.

@pleerock
Copy link
Member

pleerock commented Oct 6, 2020

HMM, one of the mainline drivers don't support it either but I'm not sure which one.

It would be great I think to quickly understand from CI what exact driver failed, to improve our developer experience.

@imnotjames
Copy link
Contributor Author

Definitely - a big reason why it's difficult is because of how we've structured our tests, with the connections promises.

@imnotjames imnotjames force-pushed the feat/ilike branch 3 times, most recently from 91b9872 to 7dd5426 Compare October 7, 2020 02:12
the next branch has support for ilike and this backports
the FindOperator into the main branch
@pleerock pleerock merged commit c8bf81e into typeorm:master Oct 7, 2020
@pleerock
Copy link
Member

pleerock commented Oct 7, 2020

Thank you!

@imnotjames imnotjames added the hacktoberfest-accepted label hacktoberfest label Oct 7, 2020
zaro pushed a commit to zaro/typeorm that referenced this pull request Jan 12, 2021
the next branch has support for ilike and this backports
the FindOperator into the main branch
@imnotjames imnotjames deleted the feat/ilike branch June 21, 2021 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted label hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants