Skip to content

Short paths#752

Open
whyboris wants to merge 8 commits into
mainfrom
short-paths
Open

Short paths#752
whyboris wants to merge 8 commits into
mainfrom
short-paths

Conversation

@whyboris

@whyboris whyboris commented Apr 5, 2022

Copy link
Copy Markdown
Owner

Works for VS Code but not when you npm start :trollface:

@whyboris whyboris added help wanted Extra attention is needed ⛑️ WIP Work in progress stalled Things that need further thought, or are blocked! labels Apr 10, 2022
@whyboris

Copy link
Copy Markdown
Owner Author

Tested again today - no success, same problems as before see writeup :trollface:

🤷‍♂️ I think the service worker causes some problems I wouldn't otherwise have 🤔

@whyboris

Copy link
Copy Markdown
Owner Author

Tested again today (Angular updated several times since before), still doesn't work :trollface:

@whyboris

Copy link
Copy Markdown
Owner Author

Merged main which has Angular 20 😅 and npm start still doesn't work with @pipes shortcut :trollface:

@whyboris

Copy link
Copy Markdown
Owner Author

I wonder if it's never meant to work: https://www.typescriptlang.org/tsconfig/#paths

Note that this feature does not change how import paths are emitted by tsc, so paths should only be used to inform TypeScript that another tool has this mapping and will use it at runtime or when bundling.

@whyboris

Copy link
Copy Markdown
Owner Author

Link for research: angular/angular-cli#1465 (comment)

Looks like you need baseUrl too.
Probably need to add this to all the tsconfig.---.json files in the repository. I tried many variations but nothing works still 🤦

@whyboris

Copy link
Copy Markdown
Owner Author

🤷 frustrating -- still doesn't work. I'll keep the PR open in case someone figures something out 🤞

I'll re-visit this in 2027 (or earlier if someone comes up with a solution ❤️ ) 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Extra attention is needed stalled Things that need further thought, or are blocked! ⛑️ WIP Work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant