Skip to content

Use hash of test files instead of unique ID - #102

Merged
SimonRit merged 1 commit into
RTKConsortium:mainfrom
acoussat:test-files-hash
Oct 5, 2026
Merged

SimonRit merged 1 commit into
RTKConsortium:mainfrom
acoussat:test-files-hash

Conversation

@acoussat

Copy link
Copy Markdown
Collaborator

Unique ID does not change when content of file changes, unlike the hash. Using the hash is more robust as it allows several versions or branches of PCT to use their own test files.

@SimonRit SimonRit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. I suppose the failing Python test is not caused by this PR?

Unique ID does not change when content of file changes, unlike the hash.
Using the hash is more robust as it allows several versions or branches of PCT to use their own test files.
@acoussat

acoussat commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator Author

LGTM. I suppose the failing Python test is not caused by this PR?

It was, I had completely messed up the test data last week. I fixed it now!

@SimonRit
SimonRit merged commit 18aab6d into RTKConsortium:main Oct 5, 2026
13 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.

2 participants