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: add type declarations for configs #347

Merged
merged 5 commits into from
Dec 6, 2024
Merged

feat: add type declarations for configs #347

merged 5 commits into from
Dec 6, 2024

Conversation

G-Rath
Copy link
Contributor

@G-Rath G-Rath commented Nov 28, 2024

Since the flat configuration system has us doing native JavaScript imports, TypeScript can now do proper type checking which means it's useful to start shipping types.

Because TypeScript requires file names to be unique (of which the file extension is not part of), I've introduced a single declaration file that holds the types for each config, rather than having a .d.ts per config, and to ensure maximum compatibility I've defined declarations for each configuration both with and without the .js extension.

@G-Rath G-Rath merged commit 7d1577d into main Dec 6, 2024
18 checks passed
@G-Rath G-Rath deleted the add-types branch December 6, 2024 00:34
github-actions bot pushed a commit that referenced this pull request Dec 6, 2024
Copy link

github-actions bot commented Dec 6, 2024

🎉 This PR is included in version 3.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants