-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
🌱 Cleanup special handling for tilt_modules folder #7774
🌱 Cleanup special handling for tilt_modules folder #7774
Conversation
Signed-off-by: Chirayu Kapoor <[email protected]>
Thanks for picking up the issue cc @oscr who did some work in this area recently |
Based on the docs: https://docs.tilt.dev/extensions.html#a-note-on-upgrading-from-old-versions Great to see a cleanup. Great catch @chiukapoor! /lgtm |
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.
Awesome! Thanks for the cleanup
/lgtm
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
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabriziopandini The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Chirayu Kapoor [email protected]
What this PR does / why we need it:
Clean up special handling for the tilt_modules folder as there is no more tilt_modules folder in the project.
Which issue(s) this PR fixes
Fixes #7773