Skip to content

Conversation

@andrsd
Copy link
Collaborator

@andrsd andrsd commented Apr 9, 2024

  • Adding SPDX tags into source files
  • gha: updating QA workflow to check for missing SPDX tags in source files

@andrsd andrsd self-assigned this Apr 9, 2024
@andrsd andrsd requested review from ragusa and wdhawkins April 9, 2024 13:49
@andrsd
Copy link
Collaborator Author

andrsd commented Apr 9, 2024

After this PR is merged you will need to include the SPDX tags into every new .cc and .h file. It is just 2 lines:

// SPDX-FileCopyrightText: 2024 The OpenSn Authors <https://open-sn.github.io/opensn/>
// SPDX-License-Identifier: MIT

The year should be the year when the file was created. So, for the rest of the year you can just copy and paste this.

There is a check in place that will make sure files are not missing the tags. It is called "qa / check spdx tags". So, if you see that failing, it means you forgot to include the above notice. The check will tell you which files are missing the tags, if you go look at the details/log.

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