Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include file path when generating file hash #4821

Merged
merged 17 commits into from
Jun 10, 2024
Merged

Include file path when generating file hash #4821

merged 17 commits into from
Jun 10, 2024

Conversation

timkpaine
Copy link
Contributor

@timkpaine timkpaine commented Mar 23, 2023

Description

Include file contents when generating file hash, in case different files have the same contents.

Fixes: #4814
Xref: conda-forge/staged-recipes#22323

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?

I'm not sure how / where to test this, please advise.

  • Add / update outdated documentation?

@conda-bot
Copy link
Contributor

We require contributors to sign our Contributor License Agreement and we don't have one on file for @timkpaine.

In order for us to review and merge your code, please e-sign the Contributor License Agreement PDF. We then need to manually verify your signature. We will ping the bot to refresh the PR status when we have confirmed your signature.

@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label May 10, 2023
@jezdez jezdez requested a review from a team January 25, 2024 09:08
@jezdez
Copy link
Member

jezdez commented Jan 25, 2024

@conda/builds-tools Any suggestion where @timkpaine could add a test for this?

@jezdez jezdez changed the title include filename in hash, fixes #4814 Include file contents when generating file hash Jan 25, 2024
Copy link

codspeed-hq bot commented Mar 18, 2024

CodSpeed Performance Report

Merging #4821 will not alter performance

Comparing timkpaine:tkp/4814 (312c888) with main (01b47f4)

Summary

✅ 3 untouched benchmarks

@isuruf
Copy link
Contributor

isuruf commented Mar 19, 2024

pre-commit.ci autofix

tests/test_subpackages.py Outdated Show resolved Hide resolved
tests/test_post.py Outdated Show resolved Hide resolved
@isuruf
Copy link
Contributor

isuruf commented Apr 17, 2024

Ready for review @conda/builds-tools

@isuruf isuruf changed the title Include file contents when generating file hash Include file path when generating file hash Jun 3, 2024
jaimergp
jaimergp previously approved these changes Jun 3, 2024
@isuruf isuruf merged commit e38e7d6 into conda:main Jun 10, 2024
28 checks passed
@beeankha beeankha mentioned this pull request Jul 16, 2024
55 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed [bot] added once the contributor has signed the CLA
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Build seems to be caching overlinking check information
5 participants