Skip to content

Conversation

@jgangel
Copy link
Contributor

@jgangel jgangel commented Jan 12, 2023

This includes modules, state and their corresponding unit tests for content libraries

@jgangel jgangel changed the title Content library only [CoE] Content libraries Jan 12, 2023
Copy link
Contributor

@dmurphy18 dmurphy18 left a comment

Choose a reason for hiding this comment

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

quick glance over the files, and noted we need to get the copyright and license statements in there. This probably needs to be done for all files to be compliant with the Open Source Office.

@@ -0,0 +1,127 @@
# Copyright 2021 VMware, Inc.
Copy link
Contributor

Choose a reason for hiding this comment

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

2021-2023

@@ -0,0 +1,91 @@
# SPDX-License-Identifier: Apache-2.0
Copy link
Contributor

Choose a reason for hiding this comment

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

missing copyright

@@ -0,0 +1,116 @@
"""
Copy link
Contributor

Choose a reason for hiding this comment

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

copyright and license

@@ -0,0 +1,155 @@
"""
Copy link
Contributor

Choose a reason for hiding this comment

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

copyright and license

@jgangel jgangel force-pushed the content-library-only branch from c6e59a9 to 3f5224c Compare January 19, 2023 14:03
Copy link
Contributor

@waynew waynew left a comment

Choose a reason for hiding this comment

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

Haven't checked the rest of this PR yet, but it does need a changelog 👍

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants