-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[Concept Entry] SIMILAR TO in SQL #5209
Conversation
Hi @cigar-galaxy82, I have made the required changes in my latest commit, could you please approve to initialize the tests. Regards, |
Hi @saldanhad thank you for contributing to Codecademy docs, I have suggested a few changes please modify them. |
Hi @cigar-galaxy82, thanks for your review, suggested changes are implemented. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for merge, I did some description changes and few first person speech issues were there, thank you for the entry :)
Thanks for the reviews, fixed linting, PR ready for merge. |
@saldanhad we are waiting for the 1st reviewer's approval and thank u for the fix :) |
@cigar-galaxy82, @dakshdeepHERE Could one of you please merge the PR, Thanks! |
👋 @saldanhad 🎉 Your contribution(s) can be seen here: https://www.codecademy.com/resources/docs/sql/operators/similar-to Please note it may take a little while for changes to become visible. |
Thank you for the contribution @saldanhad :) |
Description
Added documentation for SIMILAR TO sql operator.
Issue Solved
Closes #5165
Type of Change
Checklist
main
branch.Issues Solved
section.