[HfFileSystem] Copy non lfs files

[lhoestq]

Added CommitOperationCopy support for non-lfs files.

This test already exists to check that copying works with hffs:

huggingface_hub/tests/test_hf_file_system.py

Lines 188 to 194 in 8a9854f

	def test_copy_file(self):
	# Non-LFS file
	self.assertIsNone(self.hffs.info(self.hf_path + "/data/text_data.txt")["lfs"])
	self.hffs.cp_file(self.hf_path + "/data/text_data.txt", self.hf_path + "/data/text_data_copy.txt")
	with self.hffs.open(self.hf_path + "/data/text_data_copy.txt", "r") as f:
	self.assertEqual(f.read(), "dummy text data")
	self.assertIsNone(self.hffs.info(self.hf_path + "/data/text_data_copy.txt")["lfs"])

I simply removed the condition in hffs to use CommitOperationCopy even for non-lfs files.

Close huggingface/dataset-viewer#2317

[HuggingFaceDocBuilderDev]

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

[Wauplin]

Thanks for the PR @lhoestq :) I would prefer not to rely on the fsspec overhead when downloading the regular files since a simple http request is also fine. All comments are about making this change.

As an additional change, could you update the docstring of CommitOperationCopy now that non-LFS copies are enabled on the repo. Otherwise, looks good to me :) Glad that the testing part was that easy :D Thanks in advance!

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (c524a86) 82.39% compared to head (1656953) 82.32%.

Files	Patch %	Lines
src/huggingface_hub/_commit_api.py	93.33%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1996      +/-   ##
==========================================
- Coverage   82.39%   82.32%   -0.07%     
==========================================
  Files          66       66              
  Lines        8240     8247       +7     
==========================================
  Hits         6789     6789              
- Misses       1451     1458       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Wauplin]

Thanks for making the changes @lhoestq! Ready to be merged :)