Skip to content
This repository was archived by the owner on Mar 6, 2024. It is now read-only.

Update Readme to add components package release version matrix #307

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update Readme to add components package release version matrix #307

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,3 +99,10 @@ To get further help, please file issues on github.

To get more help on the Angular CLI use `ng help` or go check out the
[Angular CLI README](https://github.com/angular/angular-cli/blob/master/README.md).

## Components package versions corresponding to VMware cloud director(VCD) application releases

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also put the version of Clarity we require? I know our package.json in components isn't perfectly up to date.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should and we should fix the package.json to have the correct dependencies

Copy link
Contributor Author

@ps37 ps37 Feb 5, 2021 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Following are my thoughts:

  • As our components use Clarity, we should list @clr/ packages in the peer dependencies list of our components project
  • We should then also fix the versions of @angular/ packages listed there. They should be updated to angular versions with which the above-listed Clarity version works
  • If we do the first thing above, I don't think we need to list Clarity in the version matrix

| VCD app releases | GIT tags | Components package versions |
| ------------- |:-------------:| -----:|
| 10.2.0 | components-v1.0.0 | 1.0.0 |
| 10.2.1 | components-v1.1.0 | 1.1.0 |