Skip to content

Pull request for master#1

Open
TristanCacqueray wants to merge 24 commits intojd:masterfrom
TristanCacqueray:master
Open

Pull request for master#1
TristanCacqueray wants to merge 24 commits intojd:masterfrom
TristanCacqueray:master

Conversation

@TristanCacqueray
Copy link

Discover the clone url of pagure repository
Name the remote to push with the host type

@TristanCacqueray
Copy link
Author

Ho, that's weird, my last PR was sent to the new location. Here is the log of this one:

Forked repository: https://github.com/TristanCacqueray/git-pull-request
Fetching origin
Rebasing branch `master' on branch `origin/master'
Current branch master is up to date.
Force-pushing branch `master' to remote `github'
Enumerating objects: 16, done.
Counting objects: 100% (16/16), done.
Delta compression using up to 8 threads
Compressing objects: 100% (9/9), done.
Writing objects: 100% (9/9), 1.91 KiB | 1.91 MiB/s, done.
Total 9 (delta 4), reused 0 (delta 0)
remote: Resolving deltas: 100% (4/4), completed with 3 local objects.
To https://github.com/TristanCacqueray/git-pull-request.git
   e4d50c4..c541814  master -> master
Pull-request created: https://github.com/jd/git-pull-request/pull/1

This tells git credential that the password worked and that it can be re-used.
The official git documentation gives more informations about credential.
TristanCacqueray and others added 13 commits May 8, 2019 05:08
This changes the default remote name to match the host type
so that pagure repository gets a "pagure" remote instead of
a confusing "github" one.
This fixes git pull-request with pagure hosts that use
the real user name instead of "git".
This adds back the instruction to get a valid token for the pagure API.
Signed-off-by: Julien Danjou <julien@danjou.info>
Allow to only setup the fork repo
This changes adds a filename separator to the git show command to
prevent an exception from happening when the branch name is a filename:
fatal: ambiguous argument 'name': both revision and filename
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants