-
Notifications
You must be signed in to change notification settings - Fork 118
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
refactor(normalize,ferry,ferry_generator): Add const constructor to visitors #614
refactor(normalize,ferry,ferry_generator): Add const constructor to visitors #614
Conversation
✅ Deploy Preview for ferry-gql canceled.
|
Do I need to make different pull requests for the different packages? |
for refactors like this I'm ok if it affects multiple packages, just make sure to use conventional commits with the package names so automatic versioning works |
Sure let me do that! |
7851de0
to
e619718
Compare
@knaeckeKami, I updated the commits and the title of the PR. Is it what you were expecting? |
Thanks! |
This adds a const constructor to the visitors.