Skip to content

tests: add uniform SPDX, copyright, and docstring headers (#7646) - #7721

Open
ArthPatel1502 wants to merge 1 commit into
OSGeo:mainfrom
ArthPatel1502:test-headers-7646
Open

tests: add uniform SPDX, copyright, and docstring headers (#7646)#7721
ArthPatel1502 wants to merge 1 commit into
OSGeo:mainfrom
ArthPatel1502:test-headers-7646

Conversation

@ArthPatel1502

Copy link
Copy Markdown
Contributor

Description

Addresses #7646 by implementing a uniform structural layout for the test suite files. This ensures that the test scripts comply with the project standard by including proper metadata and documentation components.

Changes

  • Added standard SPDX-License-Identifier: GPL-2.0-or-later headers.
  • Integrated the GRASS Development Team copyright block.
  • Added descriptive, module-level docstrings to temporal/t.connect/tests/test_t_connect.py.

Fixes

Fixes #7646

@github-actions github-actions Bot added temporal Related to temporal data processing Python Related code is in Python module tests Related to Test Suite labels Jul 15, 2026
@@ -1,3 +1,8 @@
# SPDX-License-Identifier: GPL-2.0-or-later
# COPYRIGHT: (C) 2026 by the GRASS Development Team
# This program is free software under the GNU General Public License

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This needs to be removed as there is now "SPDX-License-Identifier: ..."

@@ -1,3 +1,8 @@
# SPDX-License-Identifier: GPL-2.0-or-later
# COPYRIGHT: (C) 2026 by the GRASS Development Team

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This needs to become "SPDX-FileCopyrightText: ..." (and no "(C)").
We are currently working out the style, so it might be best to wait for it.

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

Labels

module Python Related code is in Python temporal Related to temporal data processing tests Related to Test Suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] tests: Tests should have proper headers

2 participants