Ignore Circular Dependencies
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 10.1k
- Forks
- 413
- Avg merge
- 1d 10h
- Merged PRs (30d)
- 6
Description
Describe the Feature You Want
When loading or including an extra Tilt file, add a param where we can tell just to ignore when a circular dependency happens. So, when it happens, Tilt just doesn't load the file again, everything is fine.
Current Behavior
It throws an error when circular dependency.
Why Do You Want This?
The company has very intertwined situations, and some lead to circular dependencies. Currently, it shows as an error. Yes, it's not the ideal scenario, but it doesn't need to be an error.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing Tilt's extra-file loading or include entry point and the existing circular-dependency error path. The opt-in behavior should skip a circular include without raising an error or loading the file again, while the current error behavior remains unchanged when the option is not enabled.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100