Skip to content

ci: update pre-commit config#291

Closed
Alchez wants to merge 10 commits intoversion-15from
feat-test-utils
Closed

ci: update pre-commit config#291
Alchez wants to merge 10 commits intoversion-15from
feat-test-utils

Conversation

@Alchez
Copy link
Copy Markdown
Collaborator

@Alchez Alchez commented Mar 25, 2025

Closes #242

@Alchez Alchez requested a review from agritheory March 25, 2025 10:35
Repository owner deleted a comment from github-actions Bot Apr 1, 2025
@Alchez Alchez force-pushed the feat-test-utils branch from 4658d34 to 1f9b4d1 Compare April 1, 2025 11:23
@Alchez
Copy link
Copy Markdown
Collaborator Author

Alchez commented Apr 1, 2025

@agritheory The tests seem to be failing at an upstream class function, but I'm not sure how to resolve it. It happens when trying to submit a Payment Entry in the test setup.

@Alchez Alchez force-pushed the feat-test-utils branch from 918fbd0 to 61b7ec0 Compare April 2, 2025 05:27
Repository owner deleted a comment from github-actions Bot Apr 2, 2025
Repository owner deleted a comment from github-actions Bot Apr 2, 2025
Repository owner deleted a comment from github-actions Bot Apr 2, 2025
Repository owner deleted a comment from github-actions Bot Apr 2, 2025
@Alchez
Copy link
Copy Markdown
Collaborator Author

Alchez commented Apr 2, 2025

@agritheory I've added the Track Overrides action, and upstream commit references for Payment Entry from the time the functions were added into this app.

@agritheory
Copy link
Copy Markdown
Owner

@Alchez We don't actually want the "historic" commit hashes for the functions but the most recent "approved" ones.

@agritheory
Copy link
Copy Markdown
Owner

agritheory commented Apr 15, 2025

@Alchez Please give this one a rebase. New track overrides have been added.

@Alchez
Copy link
Copy Markdown
Collaborator Author

Alchez commented Apr 17, 2025

@agritheory Okay, I've rebased with version-15 so this one is ready for review now.

Repository owner deleted a comment from github-actions Bot Jun 12, 2025
@github-actions
Copy link
Copy Markdown
Contributor

❌ Code Duplication Report - Needs attention

Metric Value
Total Clones 1
Duplicated Lines 71
Duplication % 272.00%

View detailed report in artifacts

How to fix duplications
  1. Extract duplicated code into reusable functions or classes
  2. Use inheritance or composition for similar structures
  3. For configuration arrays (report columns, dialog fields), add ignore comments:
/* jscpd:ignore-start */
const dialogFields = [...];
/* jscpd:ignore-end */
# jscpd:ignore-start
columns = [...]
# jscpd:ignore-end

@github-actions
Copy link
Copy Markdown
Contributor

📝 Draft Changelog Entry

Changelog Entry

This release updates the project's development and testing infrastructure. The pre-commit configuration has been updated with new checks and dependencies. Workflow files for continuous integration have been modified to streamline the testing and release processes, including updates to how pytest is executed in the CI environment and adjustments to the linting workflow.

A new workflow for overrides validation has been added. The Payment Entry configuration now includes track overrides to ensure proper financial tracking. Project dependencies have been updated in the configuration files to maintain compatibility with current tooling.

This changelog entry was automatically generated by the Changelog Generator Action.

@agritheory agritheory closed this Feb 23, 2026
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.

Migrate to agritheory/test_utils Add Track overrides to Payment Entry overrides

2 participants