refactor file headers - Githubissues

jeffmcaffer commented 6 years ago

After various discussions it was determined that the correct headers should read something like

// Copyright (c) Microsoft Corporation and others. Made available under the MIT license.
// SPDX-License-Identifier: MIT

Where the actual named copyright holder is the first legal entity to create the file. The key changes being

remove "all rights reserved"
proactively add "and others" rather than having this be some sort of thing to watch for in PRs etc.
Add an explicit callout to the license rather than just referring to the SPDX identifier.

This issue will be open for a bit for comment/feedback but we need to move pretty quickly so we'll plan to do the change in a couple days barring compelling feedback to the contrary.

@kpfleming would love to get your explicit OK here

kpfleming commented 6 years ago

No objection from me. I prefer to see the explicit reference to the license in the text, in addition to the SPDX identifier.

geneh commented 6 years ago

service PR: https://github.com/clearlydefined/service/pull/118 crawler PR: https://github.com/clearlydefined/crawler/pull/73 website PR: https://github.com/clearlydefined/website/pull/88

jeffmcaffer commented 6 years ago

all done

clearlydefined / service

refactor file headers #115