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

Fix peerDependency from ember-source #170

Merged
merged 1 commit into from
Jun 1, 2024

Conversation

mkszepp
Copy link
Contributor

@mkszepp mkszepp commented May 31, 2024

I have forgot to update the peerDependency in package.json

@mkszepp mkszepp changed the title Fix peerDependency Fix peerDependency from ember-source May 31, 2024
@mkszepp
Copy link
Contributor Author

mkszepp commented May 31, 2024

@RobbieTheWagner thanks for merging everything... do you have any plan, what we should do as next? Is there any roadmap to next major, what we should do? Or do you think that we have now everything?

Edit: in code i have seen that normalize.ts should be privat... i will add minimal fixes to this PR in next days

@RobbieTheWagner RobbieTheWagner added the breaking Breaking Change label Jun 1, 2024
@RobbieTheWagner RobbieTheWagner merged commit fb60cb8 into minutebase:master Jun 1, 2024
13 checks passed
@RobbieTheWagner
Copy link
Collaborator

@mkszepp no plans, I think updating all this is probably all we need.

@mkszepp mkszepp deleted the patch-1 branch June 1, 2024 11:28
@mkszepp
Copy link
Contributor Author

mkszepp commented Jun 1, 2024

@RobbieTheWagner okay that sound good... the only thing what i would like to do, before we are making the release is, exporting the normalize util private. I will look to make a PR in next days

@RobbieTheWagner
Copy link
Collaborator

Sounds good @mkszepp!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking Change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants