Skip to content

feat: backport ilike from next#6862

Merged
pleerock merged 1 commit intotypeorm:masterfrom
imnotjames:feat/ilike
Oct 7, 2020
Merged

feat: backport ilike from next#6862
pleerock merged 1 commit intotypeorm:masterfrom
imnotjames:feat/ilike

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.

2 participants