Skip to content
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

feat: infer aliases from tsconfig.json #494

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from
Draft
Prev Previous commit
Next Next commit
update
kricsleo committed Jan 25, 2025

Verified

This commit was signed with the committer’s verified signature.
apoelstra Andrew Poelstra
commit 0b9a05d7d824d67ac25ffd6241cb49418635d996
2 changes: 1 addition & 1 deletion src/builders/rollup/utils.ts
Original file line number Diff line number Diff line change
@@ -40,7 +40,7 @@ export async function resolveAliases(ctx: BuildContext): Promise<Record<string,

/**
* REVIEW: This makes alias resolution asynchronous (which is contagious),
* because we are lazy loading TypeScript,
* because we are dynamic loading TypeScript (cause it's a peer dependency),
* or we can use a synchronous alternative [get-tsconfig](https://github.com/privatenumber/get-tsconfig).
*
* Additionally, do we need a flag to explicitly enable this feature?