Skip to content
This repository has been 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

Conversation

ps37
Copy link
Contributor

@ps37 ps37 commented Feb 3, 2021

Add a table showing Components package versions corresponding to VMware cloud director(VCD) application releases

PR Checklist

Please check if your PR fulfills the following requirements:

  • [ N/A] Tests for the changes have been added (for bug fixes/features)
  • [N/A ] Examples have been added / updated (for bug fixes / features)
  • [N/A ] Changelog has been updated

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • [ X ] Documentation content changes
  • Example website changes
  • Version bump
  • Other... Please describe:

What does this change do?

Update readme to add a table showing Components package versions corresponding to VMware cloud director(VCD) application releases

Screen Shot 2021-02-03 at 8 10 14 AM

@vmwclabot
Copy link

@ps37, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <[email protected]> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

@codecov
Copy link

codecov bot commented Feb 3, 2021

Codecov Report

Merging #307 (780b940) into master (317cfaf) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #307   +/-   ##
=======================================
  Coverage   88.68%   88.68%           
=======================================
  Files          93       93           
  Lines        2855     2855           
  Branches      437      437           
=======================================
  Hits         2532     2532           
  Misses        203      203           
  Partials      120      120           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 317cfaf...c3078a8. Read the comment docs.

@ps37 ps37 changed the title Update Readme Update Readme to add components package release version matrix Feb 3, 2021
…ponding to VCD app releases

Add a table showing Components package versions corresponding to VMware cloud director(VCD) application releases

Signed-off-by: ps37 <[email protected]>
@@ -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

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants