Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated thirdpartynotice file #613

Merged
merged 2 commits into from Oct 10, 2019
Merged

Conversation

rpohane
Copy link
Contributor

@rpohane rpohane commented Sep 26, 2019

Added missing dependencies from go.mod file to our third party notice file

Copy link
Contributor

@zezha-msft zezha-msft left a comment

Choose a reason for hiding this comment

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

:shipit:

@zezha-msft
Copy link
Contributor

@JohnRusk could you please review/merge? It's much easier to see the changes (without the spacing adjustments) in Codeflow.

Copy link
Member

@JohnRusk JohnRusk left a comment

Choose a reason for hiding this comment

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

There are several things which we are using but which are not included in this file. We need to add them to the file. Conveniently, all are using licences that are already present in the file. they are:

cpuguy83/go-md2man
stretchr/objx
gopkg.in/ini.v1 v1.42.0
gopkg.in/yaml.v2 v2.2.2

ThirdPartyNotice.txt Outdated Show resolved Hide resolved
@JohnRusk JohnRusk merged commit 2782246 into Azure:master Oct 10, 2019
JohnRusk added a commit that referenced this pull request Oct 10, 2019
JohnRusk added a commit that referenced this pull request Oct 10, 2019
(Cherry picked to dev after PR accidently had master as base)
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.

None yet

4 participants