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

chore(require-hook): misc updates #960

Merged
merged 4 commits into from Oct 17, 2021
Merged

chore(require-hook): misc updates #960

merged 4 commits into from Oct 17, 2021

Conversation

G-Rath
Copy link
Collaborator

@G-Rath G-Rath commented Oct 16, 2021

(I'm a little torn between using docs and chore)

These are a few cases that we weren't explicitly testing for, plus I realised that classes should be allowed which they are but not included in the documented list of things we allow.

@SimenB SimenB merged commit 778bd21 into main Oct 17, 2021
4 checks passed
@SimenB SimenB deleted the require-hook-updates branch Oct 17, 2021
@github-actions
Copy link

@github-actions github-actions bot commented Oct 17, 2021

🎉 This PR is included in version 25.2.2 🎉

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants