Add GetFileReader() & GetFile() support git-lfs #595

Merged
6543 merged 4 commits from getFile_supportLFS_and_add_GetFileReader into master 2022-07-29 10:51:06 +00:00
Owner

Add GetFileReader that acts like GetFile but dont read into a buffer but return a ReadCloser instead & add lfs support

Add GetFileReader that acts like GetFile but dont read into a buffer but return a ReadCloser instead & add lfs support
6543 added this to the v0.16.0 milestone 2022-07-27 12:52:17 +00:00
6543 added the
kind/feature
label 2022-07-27 12:52:17 +00:00
6543 added 1 commit 2022-07-27 12:52:17 +00:00
Add GetFileReader that acts like GetFile but dont read into a buffer but return a ReadCloser instead & add lfs support
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is failing
e9f4f96157
lunny approved these changes 2022-07-27 13:55:17 +00:00
Owner

CI failure is related.

CI failure is related.
Author
Owner

likely due to reuse and a missing check ...

... will fix it

likely due to reuse and a missing check ... ... will fix it
6543 added 2 commits 2022-07-29 00:38:31 +00:00
enhance tests
All checks were successful
continuous-integration/drone/pr Build is passing
continuous-integration/drone/push Build is passing
431f1b6bc1
Author
Owner

done

done
KN4CK3R reviewed 2022-07-29 06:37:11 +00:00
@ -39,10 +39,12 @@ func TestGetGlobalSettings(t *testing.T) {
attachSettings, _, err := c.GetGlobalAttachmentSettings()
assert.NoError(t, err)
if assert.True(t, len(attachSettings.AllowedTypes) > 100) {
Member

Why is this needed?

Why is this needed?
Author
Owner

since default values have changed we get this: https://drone.gitea.com/gitea/go-sdk/575/1/3

this check does only check if there are enough vars but not fail if a single bit will change again

since default values have changed we get this: https://drone.gitea.com/gitea/go-sdk/575/1/3 this check does only check if there are enough vars but not fail if a single bit will change again
Member

Changed to assert.NotEmpty

Changed to `assert.NotEmpty`
KN4CK3R marked this conversation as resolved
6543 added 1 commit 2022-07-29 10:22:06 +00:00
nit
Some checks failed
continuous-integration/drone/push Build is failing
continuous-integration/drone/pr Build is passing
e0dd72c9f9
KN4CK3R approved these changes 2022-07-29 10:23:42 +00:00
6543 merged commit cc14c63ccc into master 2022-07-29 10:51:06 +00:00
6543 deleted branch getFile_supportLFS_and_add_GetFileReader 2022-07-29 10:51:06 +00:00
Sign in to join this conversation.
No description provided.