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

Strictly manage constants and improve types #11101

Merged
merged 2 commits into from
Oct 26, 2023

Conversation

ssi02014
Copy link
Contributor

@ssi02014 ssi02014 commented Oct 26, 2023

@bluebill1049 👋
I thought it would be nice to have stricter control over constants, so I've added "as const" and improved the types accordingly.

I'd appreciate it if you could review if this is appropriate! 🙏

@codesandbox
Copy link

codesandbox bot commented Oct 26, 2023

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@ssi02014 ssi02014 changed the title Improve constants type Improve constants and type Oct 26, 2023
@ssi02014 ssi02014 changed the title Improve constants and type Improve constants type Oct 26, 2023
@ssi02014 ssi02014 changed the title Improve constants type Strictly manage constants and improve types Oct 26, 2023
Copy link
Member

@bluebill1049 bluebill1049 left a comment

Choose a reason for hiding this comment

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

LGTM

@bluebill1049 bluebill1049 merged commit a7ee911 into react-hook-form:master Oct 26, 2023
7 checks passed
@ssi02014 ssi02014 deleted the imp/types branch October 27, 2023 00:12
@ssi02014
Copy link
Contributor Author

@bluebill1049 Thanks 😃

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.

None yet

2 participants