Skip to content

chore: add SPDX-License-Identifier headers to Python scripts#14

Merged
mmilanez merged 5 commits into
mmilanez:mainfrom
leadsolutions-mike:chore/spdx-headers
Jun 2, 2026
Merged

chore: add SPDX-License-Identifier headers to Python scripts#14
mmilanez merged 5 commits into
mmilanez:mainfrom
leadsolutions-mike:chore/spdx-headers

Conversation

@leadsolutions-mike

Copy link
Copy Markdown
Contributor

Summary

Closes #7.

Adds # SPDX-License-Identifier: Apache-2.0 to the five Python scripts in .agents/scripts/ that were missing the header:

  • validate_state.py — after the shebang line
  • migrate_to_v2.py — after the shebang line
  • test_validate_state.py — at the top of the file
  • test_migrate_to_v2.py — at the top of the file
  • conftest.py — at the top of the file

No functional changes. This brings the scripts into compliance with the Apache 2.0 license declared in LICENSE and makes them compatible with SPDX-aware license scanners (FOSSA, Snyk License, etc.).

Test plan

  • CI passes (state-validation workflow)
  • Verify SPDX header is present in each of the 5 files
  • Confirm no functional change: pytest .agents/scripts/ -v produces the same results as before

Contributed by @leadsolutions-mike as external collaborator.

@mmilanez mmilanez left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Reviewed the diff and validation results. The PR only adds SPDX-License-Identifier headers to the expected Python scripts, satisfies #7, and both required checks pass.

@mmilanez mmilanez merged commit 74a5e14 into mmilanez:main Jun 2, 2026
2 checks passed
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.

chore: add SPDX headers to framework Python scripts

2 participants