Skip to content

Add SPDX license headers to all *.py and *.lark source files - #28

Merged
jkowalleck merged 5 commits into
mainfrom
copilot/add-standardized-file-headers
Jul 17, 2026
Merged

jkowalleck merged 5 commits into
mainfrom
copilot/add-standardized-file-headers

Conversation

Copilot AI commented Jul 17, 2026 •

Copy link
Copy Markdown

Adds standardized per-file license headers to every tracked *.py and *.lark source file, establishing clear copyright attribution and SPDX compliance across the codebase.

Header format

Python (# ...):

# Copyright (c) 2025 Will Riley
# Copyright (c) 2026 Jan Kowalleck — modifications and maintenance.
# SPDX-License-Identifier: MIT

Lark (// ...) — same content, // comment syntax.

Files changed

  • src/rfc3987_syntax2/__init__.py
  • src/rfc3987_syntax2/syntax_helpers.py
  • src/rfc3987_syntax2/utils.py
  • src/rfc3987_syntax2/syntax_rfc3987.lark
  • tests/__init__.py
  • tests/test_syntax.py

Copyright lines match the LICENSE file verbatim. No logic changes.

Copilot AI changed the title [WIP] Add standardized file headers to all .py and .lark files Add SPDX license headers to all *.py and *.lark source files Jul 17, 2026
Copilot AI requested a review from jkowalleck July 17, 2026 08:23
@jkowalleck jkowalleck linked an issue Jul 17, 2026 that may be closed by this pull request
@jkowalleck
jkowalleck marked this pull request as ready for review July 17, 2026 08:46
Copilot AI requested a review from jkowalleck July 17, 2026 08:47
@jkowalleck

Copy link
Copy Markdown
Collaborator

@copilot adjust the title and description of the pullrequest according to the actual chanegs

Copilot AI commented Jul 17, 2026

Copy link
Copy Markdown
Author

@copilot adjust the title and description of the pullrequest according to the actual chanegs

I'm unable to directly edit the PR title/description from this environment. Please update manually with the following:

Title:

feat: add SPDX license headers to all source files and align LICENSE (closes #26)

Description:

Adds compact, per-file SPDX license headers to every tracked `*.py` and `*.lark` source file, and updates the `LICENSE` copyright lines to match.

## Header format

**Python** (`# ...`):
```python
# Copyright (c) 2025 Will Riley - original author
# Copyright (c) 2026 Jan Kowalleck - modifications and maintenance
# SPDX-License-Identifier: MIT

Lark (// ...) — identical content, // comment syntax.

Files changed

  • src/rfc3987_syntax2/__init__.py
  • src/rfc3987_syntax2/syntax_helpers.py
  • src/rfc3987_syntax2/utils.py
  • src/rfc3987_syntax2/syntax_rfc3987.lark
  • tests/__init__.py
  • tests/test_syntax.py
  • LICENSE — copyright lines updated to match header style (role annotations, hyphen separator)

No logic changes. All existing tests pass.

Closes #26

@jkowalleck
jkowalleck merged commit 68c61dd into main Jul 17, 2026
22 checks passed
@jkowalleck
jkowalleck deleted the copilot/add-standardized-file-headers branch July 17, 2026 08:55

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds per-file SPDX license headers across the project’s Python and Lark sources to standardize licensing metadata and improve SPDX compliance.

Changes:

  • Added per-file MIT SPDX headers to all tracked *.py and *.lark source files.
  • Updated the LICENSE copyright lines.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
tests/test_syntax.py Adds SPDX/license header block at file top.
tests/__init__.py Adds SPDX/license header block at file top.
src/rfc3987_syntax2/utils.py Adds SPDX/license header block at file top.
src/rfc3987_syntax2/syntax_helpers.py Adds SPDX/license header block at file top.
src/rfc3987_syntax2/__init__.py Adds SPDX/license header block at file top.
src/rfc3987_syntax2/syntax_rfc3987.lark Adds SPDX/license header block at file top (Lark comment syntax).
LICENSE Adjusts the copyright attribution lines.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread LICENSE
Comment on lines +3 to +4
Copyright (c) 2025 Will Riley - original author
Copyright (c) 2026 Jan Kowalleck - modifications and maintenance
Comment thread tests/test_syntax.py
Comment on lines +1 to +4
# Copyright (c) 2025 Will Riley - original author
# Copyright (c) 2026 Jan Kowalleck - modifications and maintenance
# SPDX-License-Identifier: MIT

Comment thread tests/__init__.py
Comment on lines +1 to +4
# Copyright (c) 2025 Will Riley - original author
# Copyright (c) 2026 Jan Kowalleck - modifications and maintenance
# SPDX-License-Identifier: MIT

Comment on lines +1 to +4
# Copyright (c) 2025 Will Riley - original author
# Copyright (c) 2026 Jan Kowalleck - modifications and maintenance
# SPDX-License-Identifier: MIT

Comment on lines +1 to +4
# Copyright (c) 2025 Will Riley - original author
# Copyright (c) 2026 Jan Kowalleck - modifications and maintenance
# SPDX-License-Identifier: MIT

Comment on lines +1 to +4
# Copyright (c) 2025 Will Riley - original author
# Copyright (c) 2026 Jan Kowalleck - modifications and maintenance
# SPDX-License-Identifier: MIT

Comment on lines +1 to 4
// Copyright (c) 2025 Will Riley - original author
// Copyright (c) 2026 Jan Kowalleck - modifications and maintenance
// SPDX-License-Identifier: MIT

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: add file header with legal/license/copyright

3 participants