Skip to content

Core - update : angular v16 #100

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

Merged
merged 6 commits into from
Jun 15, 2023
Merged

Core - update : angular v16 #100

merged 6 commits into from
Jun 15, 2023

Conversation

rbalet
Copy link
Contributor

@rbalet rbalet commented May 24, 2023

Description

I've done multiple commit to split every change I made, but I'll comment them here.

Done

  1. Update up to angular v16.0.2
  2. Update every dependencies to their latest stable version
  3. Remove unused tsickle library It wasn't used anymore and was quite old
  4. Add a new script npm run publish to make life easier
  5. Add a new configuration --production to change the compilation mode -> package get reduced from 207kb down to 176kb
  6. Update the library from 1.6.0 to 1.7.0

Additional

I've tested it on my angular v16 project, works like a charm.

But a second control is never bad

Resolve

The following shall be closed, since everything is contained in this release

@axi92
Copy link

axi92 commented May 30, 2023

Nice one! pls merge it if you have time =) @AlexMiniApps

@drakedeatonuk
Copy link

FYI guys this branch is also working well for me on Angular 16 w/ Ionic 7

@Ice32
Copy link

Ice32 commented Jun 6, 2023

Is this repo maintained? Is there a published fork with Angular 16 support?

@rbalet
Copy link
Contributor Author

rbalet commented Jun 6, 2023

@Ice32 Yes it is and I had personal contact with the maintainer, @AlexMiniApps.

He's just a bit in a rush at the moment and couldn't find time to merge my request.

That said, if you have only one project, you could download my fork locally , rename the lib folder to angular-core-input and use it directly.

You may then have to npm i some dependencies

@Wahsner
Copy link

Wahsner commented Jun 6, 2023

Hopefully this will be merged soon. Otherwise going to have to remove the package from project as it's blocking a release.

@tortuc
Copy link

tortuc commented Jun 15, 2023

Le me know when will it be merged. Because I stopped using this package in my project because this couldn't be used in Angular 16.

@JeroenBoesten
Copy link

We also would like to see this PR merged 👍

@AlexMiniApps AlexMiniApps merged commit f8ef761 into AlexMiniApps:master Jun 15, 2023
@JeroenBoesten
Copy link

@AlexMiniApps Thanks for merging, when can we expect the new release containing the changes?

@tortuc
Copy link

tortuc commented Jun 19, 2023

@AlexMiniApps
Thank you very much for merging.

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.

8 participants