Skip to content

Conversation

@BobbyRBruce
Copy link
Member

No description provided.

@BobbyRBruce BobbyRBruce requested a review from Copilot May 27, 2025 13:06
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR performs a final tidying pass on the repository, updating license headers, formatting, and workflow configurations while ensuring consistency across the codebase.

  • Updated license and copyright headers
  • Standardized import formatting and string quoting
  • Adjusted GitHub Actions workflows and pre-commit configuration for improved consistency

Reviewed Changes

Copilot reviewed 22 out of 22 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
update_filter_view.py Added/upated copyright headers and formatting changes
tests/resources_api_unit_tests.py Refactored multi-line formatting and improved readability
shared/utils.py, shared/database.py Standardized string formatting and minor regex adjustments
functions/* Consistent formatting and minor improvements in error messages
LICENSE Introduced placeholder text; should be updated with actual details
.pre-commit-config.yaml & workflows Updated configurations for pre-commit hooks and CI/CD workflows

LICENSE Outdated
@@ -0,0 +1,12 @@
Copyright (c) <date> <copyright holder>
Copy link

Copilot AI May 27, 2025

Choose a reason for hiding this comment

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

Replace the placeholder text with the actual copyright holder and current year to ensure the license is accurate.

Copilot uses AI. Check for mistakes.
@BobbyRBruce BobbyRBruce merged commit 19a51f7 into main Jun 4, 2025
2 checks passed
@BobbyRBruce BobbyRBruce deleted the tidyups branch June 4, 2025 21:36
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