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

Options for checking getter or setter only #694

Merged
merged 1 commit into from Feb 21, 2021

Conversation

brettz9
Copy link
Collaborator

@brettz9 brettz9 commented Feb 15, 2021

Although I know normally @gajus and @golopot did not favor options whose schemas allowed different types, given that checkGetters/checkSetters has been around as a boolean for a while, and adding a separate checkGettersWithSettesr/checkSettersWithGetters boolean option could become a confusing headache, this PR does allow a string value for checkGetters/checkSetters.

@brettz9 brettz9 force-pushed the check-getter-setter-only branch 8 times, most recently from 740f2ec to 90d02f9 Compare February 16, 2021 23:02
@brettz9 brettz9 merged commit 937d65e into gajus:master Feb 21, 2021
@brettz9 brettz9 deleted the check-getter-setter-only branch February 21, 2021 03:19
@gajus
Copy link
Owner

gajus commented Feb 21, 2021

🎉 This PR is included in version 32.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@gajus gajus added the released label Feb 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[require-jsdoc] Require JSDoc only either for the getter or the setter
3 participants