Skip to content

chore: add test for exported types in npm package #925

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

Merged
merged 3 commits into from
Dec 26, 2024
Merged

chore: add test for exported types in npm package #925

merged 3 commits into from
Dec 26, 2024

Conversation

kachick
Copy link
Contributor

@kachick kachick commented Apr 5, 2024

This PR add test for #924

Extracted from the original PR because of adding dev dependency https://github.com/arethetypeswrong/arethetypeswrong.github.io/tree/add0a9e21aee1288a13826f3cd7682840acbdc25/packages/cli


Before the change?

After the change?

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

kachick added 2 commits April 5, 2024 23:28
`npm install --save-dev "@arethetypeswrong/cli"`
Extracted from validate:ts as posttest does
@kachick kachick changed the title fix: export schema types and add the test with @arethetypeswrong/cli chore: add exported types test for npm package with @arethetypeswrong/cli Apr 5, 2024
@kachick kachick changed the title chore: add exported types test for npm package with @arethetypeswrong/cli chore: add test for exported types in npm package Apr 5, 2024
@kachick
Copy link
Contributor Author

kachick commented Dec 26, 2024

How do you think about adding this tooling?

I think adding type assertions, as these tools help with the process of publishing a package. However, I can close this PR if there are any concerns. 🙏

Copy link
Contributor

@nickfloyd nickfloyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @kachick Thank you for this change and apologies for the massive delay on getting this approved. This seems like a great addition to help us keep things "healthy" ❤️

@nickfloyd nickfloyd merged commit a3d7047 into octokit:main Dec 26, 2024
4 checks passed
@kachick
Copy link
Contributor Author

kachick commented Dec 27, 2024

Thank you! 🙏

@kachick kachick deleted the fix-missing-exporting-schema-types-with-test branch December 27, 2024 01:02
Copy link
Contributor

🎉 This PR is included in version 15.26.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