fix: Normalize relative_path and avoid emitting multiple Documents for same file #331

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Draft

varungandhi-src wants to merge 2 commits into main from vg/normalize-files

Contributor

varungandhi-src commented Mar 12, 2024

Fixes #330

Test plan

TODO:

Add test that was failing before but passes afterwards. Not 100% sure at the moment about how to add a minimal test case. We already have a multi-project test which uses a relative reference in tsconfig.json
```
snapshots/input/multi-project/packages/b/tsconfig.json
10:  "references": [{ "path": "../a" }]
```
Maybe we also need to use a relative import.

varungandhi-src added 2 commits

March 12, 2024 07:52


          WIP: Add assertion for path normalization

edddc2c


          WIP: Avoid redundant work for same path

2bb6714

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet