Skip to content

Conversation

jolheiser
Copy link
Member

Closes #21066

I believe I have this correct, as sending any data -d automatically assumes a POST request and likewise not specifying a verb should default to GET.

@jolheiser jolheiser added the type/docs This PR mainly updates/creates documentation label Sep 5, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Sep 5, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #21071 (836cc8e) into main (bb0ff77) will increase coverage by 0.44%.
The diff coverage is 53.95%.

@@            Coverage Diff             @@
##             main   #21071      +/-   ##
==========================================
+ Coverage   46.65%   47.10%   +0.44%     
==========================================
  Files        1003     1008       +5     
  Lines      136654   137718    +1064     
==========================================
+ Hits        63761    64873    +1112     
+ Misses      65066    64910     -156     
- Partials     7827     7935     +108     
Impacted Files Coverage Δ
cmd/migrate_storage.go 5.88% <0.00%> (+0.11%) ⬆️
models/repo.go 50.22% <0.00%> (+0.11%) ⬆️
models/webhook/hooktask.go 49.70% <0.00%> (-0.59%) ⬇️
modules/avatar/identicon/block.go 42.11% <ø> (ø)
modules/convert/issue_comment.go 40.60% <0.00%> (-1.26%) ⬇️
modules/git/foreachref/parser.go 94.36% <ø> (ø)
modules/git/signature_nogogit.go 61.53% <ø> (ø)
modules/log/file.go 68.78% <ø> (ø)
modules/log/smtp.go 82.35% <ø> (ø)
modules/markup/mdstripper/mdstripper.go 61.15% <ø> (ø)
... and 117 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Sep 5, 2022
@techknowlogick techknowlogick merged commit 0ad0190 into go-gitea:main Sep 5, 2022
@jolheiser jolheiser deleted the curl-usage branch September 5, 2022 12:45
zeripath pushed a commit that referenced this pull request Sep 5, 2022
zjjhot added a commit to zjjhot/gitea that referenced this pull request Sep 6, 2022
* upstream/main:
  Fix sub folder in repository missing add file dropdown (go-gitea#21069)
  [skip ci] Updated translations via Crowdin
  Add missing volume to test-e2e (go-gitea#21079)
  Fix delete user missed some comments (go-gitea#21067)
  Remove insecure flag from curl (go-gitea#21074)
  Update curl usage in API docs (go-gitea#21071)
  Move go-licenses to generate and separate generate into a frontend and backend component (go-gitea#21061)
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CURL usage is not right in https://docs.gitea.io/en-us/api-usage/
6 participants