-
Notifications
You must be signed in to change notification settings - Fork 119
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
SNOW-1491199 add ast-encoding test in precommit #2510
Conversation
95fd33e
to
28458de
Compare
please also add our team as authors to the PR :) Allows in the future folks to ping us. |
Seems like your changes contain some Local Testing changes, please request review from @snowflakedb/local-testing |
I already add code ownership. Where do you want to add author? |
in github you do co-authored-by: in the commit message. this should be done. |
Oh just saw this, will add in the followup RPs. |
Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes SNOW-1491199
Fill out the following pre-review checklist:
Please describe how your code solves the related issue.
Please write a short description of how your code change solves the related issue.
Before moving source code of IR to this repo, this PR add the test job in CI first which will run in merge gate. We will enforce this check before IR Phase 0 is complete.