Skip to content

fix: support extends . edge case#221

Merged
dominikg merged 2 commits intodominikg:mainfrom
JounQin:fix/extends_edge_cases
May 20, 2025
Merged

fix: support extends . edge case#221
dominikg merged 2 commits intodominikg:mainfrom
JounQin:fix/extends_edge_cases

Conversation

@JounQin
Copy link
Copy Markdown
Contributor

@JounQin JounQin commented May 20, 2025

close #220

@JounQin JounQin changed the title fix: support extends ., ./, .. and ../ all at once fix: support extends . edge case May 20, 2025
@dominikg
Copy link
Copy Markdown
Owner

thanks a lot. do you want to add a changeset via pnpm changeset ? i can later too but not before tomorrow.

@JounQin
Copy link
Copy Markdown
Contributor Author

JounQin commented May 20, 2025

thanks a lot. do you want to add a changeset via pnpm changeset ? i can later too but not before tomorrow.

Just curious, if you're using changeset, why https://github.com/apps/changeset-bot is not integrated?


Added.

@dominikg
Copy link
Copy Markdown
Owner

to avoid the noise it creates. i don't like automated comments and prefer less apps/integrations.

I'll add a note about it to contributing.md and PR template. thank you for your help and pointing out its not documented

@dominikg dominikg merged commit 0cc1cae into dominikg:main May 20, 2025
7 checks passed
@github-actions github-actions Bot mentioned this pull request May 20, 2025
@JounQin JounQin deleted the fix/extends_edge_cases branch May 20, 2025 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

failed to resolve "extends":"." which should resolve ./tsconfig.json

2 participants