Remove only_subdir from the vendor tool?
Nobody has claimed this yet.
- Dominant language
- SystemVerilog
- Stars
- 3.6k
- Forks
- 1.1k
- Avg merge
- 2d 22h
- Merged PRs (30d)
- 141
Description
Once PR #2077 is merged, the vendor tool will allow "mappings", which let you give a list of subdirectories that should be copied. At the moment, these are interpreted relative to the `only_subdir` key. This key is no longer strictly needed, since the path can be folded in to the mappings, so we might choose to remove it. This issue tracks that removal.
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 reviewing the vendor tool behavior described around PR #2077, especially how mappings are interpreted relative to only_subdir. Confirm that the path can be represented through mappings, then remove only_subdir and update the affected configuration or tests; done means vendor tool behavior remains correct without that key.
Written by the indexing model from the issue text.
Assessment
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100