Skip to content

Conversation

@mocsharp
Copy link
Collaborator

@mocsharp mocsharp commented Aug 16, 2022

Description

  • Add license notice on landing page & in the LICENSE file
  • Update third-party licenses for the 0.2.0 release and include license files in the Docker image.

Status

PENDING final build of Storage & Message libs

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • New tests added to cover the changes.
  • All tests passed locally.
  • Documentation comments included/updated.
  • User guide updated.
  • I have updated the changelog

Signed-off-by: Victor Chang <vicchang@nvidia.com>
Signed-off-by: Victor Chang <vicchang@nvidia.com>
Signed-off-by: Victor Chang <vicchang@nvidia.com>
@mocsharp mocsharp added the build Build & build workflow label Aug 16, 2022
@mocsharp mocsharp added this to the 0.2.0 milestone Aug 16, 2022
@mocsharp mocsharp self-assigned this Aug 16, 2022
@codecov
Copy link

codecov bot commented Aug 16, 2022

Codecov Report

Merging #115 (99db2fd) into develop (3851d32) will increase coverage by 0.02145%.
The diff coverage is n/a.

Impacted file tree graph

@@                 Coverage Diff                 @@
##             develop        #115         +/-   ##
===================================================
+ Coverage   92.87297%   92.89442%   +0.02145%     
===================================================
  Files            247         247                 
  Lines          13989       13989                 
  Branches         740         740                 
===================================================
+ Hits           12992       12995          +3     
+ Misses           779         777          -2     
+ Partials         218         217          -1     
Flag Coverage Δ
unittests 92.89442% <ø> (+0.02145%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...Gateway/Repositories/InferenceRequestRepository.cs 97.61905% <0.00000%> (+3.57143%) ⬆️

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 3851d32...99db2fd. Read the comment docs.

@mocsharp mocsharp changed the title Update license for 0.2.0 release Update third-party licenses for 0.2.0 release Aug 16, 2022
Signed-off-by: Victor Chang <vicchang@nvidia.com>
Signed-off-by: Victor Chang <vicchang@nvidia.com>
Signed-off-by: Victor Chang <vicchang@nvidia.com>
@mocsharp mocsharp requested a review from dbericat August 18, 2022 00:04
Copy link
Member

@dbericat dbericat left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Victor Chang <vicchang@nvidia.com>
Signed-off-by: Victor Chang <vicchang@nvidia.com>
@mocsharp mocsharp force-pushed the vchang/license-update branch from b28dc5d to 99db2fd Compare August 18, 2022 23:44
@mocsharp mocsharp merged commit e2c6049 into develop Aug 19, 2022
@mocsharp mocsharp deleted the vchang/license-update branch August 19, 2022 00:07
mocsharp added a commit that referenced this pull request Aug 19, 2022
* Update packages
* Update third-party-liceneses
* Include license files in Docker image
* Update CodeQL to v2
* Add license notice

Signed-off-by: Victor Chang <vicchang@nvidia.com>
@github-actions
Copy link

🎉 This issue has been resolved in version 0.2.0 🎉

The release is available on:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Build & build workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants