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.

Sign up for GitHub

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

Jump to bottom

Push to create docs #23458

Merged
merged 3 commits into from
Mar 14, 2023
Merged

Push to create docs #23458

merged 3 commits into from
Mar 14, 2023

Conversation

techknowlogick
Copy link
Member

This PR adds user friendly documentation on how to use push to create feature

@techknowlogick techknowlogick added type/docs This PR mainly updates/creates documentation skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. labels Mar 14, 2023
@techknowlogick techknowlogick added this to the 1.20.0 milestone Mar 14, 2023
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Mar 14, 2023
@jolheiser
Copy link
Member

jolheiser commented Mar 14, 2023 •
edited
Loading

A great hint (imo) is mentioning push options as well. Push-to-create is private by default (unless DEFAULT_PUSH_CREATE_PRIVATE is set to true), but you can also pass -o repo.private=false to set it in the same push as creating it.

EDIT: This already looks good (barring the relref comment from delvh), so I'd be happy to give a +1 and sending a suggestion for the above at a later time. 🙂

@techknowlogick @jolheiser
Update docs/content/doc/usage/push-to-create.en-us.md
a9a7762 
Co-authored-by: John Olheiser <john.olheiser@gmail.com>
@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 Mar 14, 2023
@jolheiser
Copy link
Member

🎺 🤖

@jolheiser jolheiser merged commit db9bb36 into go-gitea:main Mar 14, 2023
@jolheiser jolheiser changed the title docs: push to create Push to create docs Mar 14, 2023
@techknowlogick techknowlogick deleted the add-push-to-create-docs branch March 14, 2023 16:50
@jolheiser jolheiser mentioned this pull request Mar 14, 2023
Merged
lafriks pushed a commit that referenced this pull request Mar 14, 2023
@jolheiser
Push option bonus for PTC docs (#23473)
6e75739 
Follow-up for #23458 

I could have suggested this on the original PR, but I thought there
would be more to add. Hadn't noticed the push options docs already had
nearly the same shell command. 😅

Signed-off-by: jolheiser <john.olheiser@gmail.com>
zjjhot added a commit to zjjhot/gitea that referenced this pull request Mar 15, 2023
@zjjhot
Merge remote-tracking branch 'giteaofficial/main'
a4a7dce 
* giteaofficial/main: (33 commits)
  Bump webpack from 5.75.0 to 5.76.0 (go-gitea#23484)
  Replace Less with CSS (go-gitea#23481)
  Fix 'View File' button in code search (go-gitea#23478)
  Use `gitea/test_env` image instead of `golang` (go-gitea#23455)
  Skip DB tests duplicate runs on push to branches (go-gitea#23476)
  Update app.example.ini (go-gitea#23480)
  [skip ci] Updated translations via Crowdin
  Fix due date being wrong on issue list (go-gitea#23475)
  test_env: hardcode major go version in use (go-gitea#23464)
  Push option bonus for PTC docs (go-gitea#23473)
  Lint Markdown pass
  Push to create docs (go-gitea#23458)
  Convert GitHub event on actions and fix some pull_request events. (go-gitea#23037)
  Remove wrongly added column on migration test fixtures (go-gitea#23456)
  Refactor branch/tag selector to Vue SFC (go-gitea#23421)
  add admin API email endpoints (go-gitea#22792)
  add user rename endpoint to admin api (go-gitea#22789)
  Add workflow error notification in ui (go-gitea#23404)
  Make branches list page operations remember current page (go-gitea#23420)
  fix markdown lint issue (go-gitea#23457)
  ...
@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.
Reviewers

@delvh delvh delvh left review comments

@lunny lunny lunny approved these changes

@jolheiser jolheiser jolheiser approved these changes

Assignees
No one assigned
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/docs This PR mainly updates/creates documentation
Projects
None yet
Milestone
1.20.0
Development

Successfully merging this pull request may close these issues.
5 participants
@techknowlogick @jolheiser @lunny @delvh @GiteaBot