Conversation
There was a problem hiding this comment.
Code Review
The pull request adds a new update guide, which is a great addition. However, the structure of the guide could be confusing for users. The steps are not in a logical order, and it presents two different ng update commands, which could lead to errors. I've suggested a restructuring of the document to provide a clearer, step-by-step process for updating. The main points are to first update Angular, then Element, and to clarify what the automatic migration does versus what needs to be done manually.
|
Documentation. Coverage Reports: |
530dc9c to
1f4c7f5
Compare
1f4c7f5 to
773469d
Compare
spliffone
left a comment
There was a problem hiding this comment.
Can we state which @simpl/brand version is necessary, since the consumer can only guess which major version to use
773469d to
1359ceb
Compare
1359ceb to
ba232e2
Compare
Adds a general update guide for element.