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
feat(datepicker): use readonly arrays for inputs (#2773) #3424
feat(datepicker): use readonly arrays for inputs (#2773) #3424
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why specific type instead of readonly
modifier?
@Input() months: readonly MontViewModel[] = [];
I would tend to readonly
, just wondering on the choice
I just followed what was specified in the associated issue.
|
Tha'ts ok. I've updated the origin issue with description on the TS 3.4 improvements in regards of readonly types: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments
Codecov Report
@@ Coverage Diff @@
## master #3424 +/- ##
=======================================
Coverage 91.13% 91.13%
=======================================
Files 96 96
Lines 2797 2797
Branches 516 516
=======================================
Hits 2549 2549
Misses 190 190
Partials 58 58
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Before submitting a pull request, please make sure you have at least performed the following:
Part of #2773.