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(angular): v17 #82

Closed
wants to merge 6 commits into from
Closed

chore(angular): v17 #82

wants to merge 6 commits into from

Conversation

rbalet
Copy link

@rbalet rbalet commented Sep 11, 2024

Changes

  • Chore: Update the library to angular v17, (Core & Material)
  • Feat: .prettierrc to have a consistent code and avoid to many changes when fixing some issues
  • fix: init - when strings isn't set -> use a fallback in the en language
  • fix: empty strings.numbers -> Due to new typescript compilation errors
  • docs: README -> add an "how to use" for standalone users

Note

Due to the .prettierrc, a lot have being changed, to have a better look, see each commit one after the other.
I've created one commit per changes, so that you shall cherry pick if you need.

Related issue

Following shall be closed : #70

@rbalet rbalet mentioned this pull request Feb 22, 2025
@ihym
Copy link
Owner

ihym commented Feb 22, 2025

@rbalet please open a PR where you just make the changes needed for angular 17 to work.
Dont change the prettier config for example.

@rbalet
Copy link
Author

rbalet commented Feb 23, 2025

@ihym I can do, would you also like me to do the fixes in a separate pull-request?

And if I would open an issue for prettierc would you accept it? (I would do this one first then, because it make the life whole lot easier)

@ihym
Copy link
Owner

ihym commented Feb 24, 2025

Yes, open a separate PR. Dont open another issue.

@rbalet rbalet closed this Feb 24, 2025
@rbalet
Copy link
Author

rbalet commented Feb 24, 2025

Close: Will do separate commits

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.

2 participants