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

Correct copyright statements in code #3403

Merged
merged 23 commits into from
Dec 15, 2019
Merged

Correct copyright statements in code #3403

merged 23 commits into from
Dec 15, 2019

Conversation

donker
Copy link
Contributor

@donker donker commented Dec 13, 2019

This PR aims to remove all copyright statements from code files in favor of the license in the Github repo.

Copy link
Contributor

@mitchelsellers mitchelsellers left a comment

Choose a reason for hiding this comment

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

I appreciate the work here, and the existing statements should be removed, however, each file MUST have the standard heading on it. Per .NET Foundation guidelines it should be.

//
// Copyright (c) .NET Foundation. All rights reserved.
// Licensed under the MIT License. See LICENSE file in the project root for full license information.
//

@donker
Copy link
Contributor Author

donker commented Dec 13, 2019

NP, I can work with that. It seems a pity we have to bloat our code with this, but lawyers are our overlords ;-)

@donker donker changed the title Remove Copyright statements from code Correct copyright statements in code Dec 13, 2019
@donker donker added the chore label Dec 13, 2019
@mitchelsellers
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@donker
Copy link
Contributor Author

donker commented Dec 15, 2019

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@valadas
Copy link
Contributor

valadas commented Dec 15, 2019

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@valadas valadas added this to the 9.5.0 milestone Dec 15, 2019
@valadas valadas merged commit c35fdc7 into dnnsoftware:develop Dec 15, 2019
@donker donker deleted the removercr branch December 15, 2019 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants