-
Notifications
You must be signed in to change notification settings - Fork 564
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
Adds data for hooks APIs #5554
Adds data for hooks APIs #5554
Conversation
… JSON file for all hooks adds hooks build to 'build-components-json' in CI
|
👋 Hi, this pull request contains changes to the source code that github/github depends on. If you are GitHub staff, we recommend testing these changes with github/github using the integration workflow. Thanks! |
size-limit report 📦
|
You're good to just have this as "none" with no changeset, I think 👍 |
Relates to https://github.com/github/primer/issues/2986
See https://github.com/github/primer-docs/pull/206 for an example of how this will be used.
Changelog
None of the work in this PR includes user-facing changes except for a new generated JSON file with all the hooks APIs' data. Do we need a changelog for this PR?
New
Generates a JSON file with hooks APIs' data in
generated/
.Changed
Removed
Rollout strategy
Testing & Reviewing
To test, run
npm run build:hooks.json
and confirm that there are no errors.To review:
packages/react/script/hooks-json/build.ts
Merge checklist