Як змяніць запыт GitHub цягнуць?

Я адкрыў запыт цягнуць да праекту. Суправаджалы вырашыў прыняць яго, але сказаў мне, каб змяніць некаторыя змесціва.

Як я магу гэта зрабіць? Ці павінен я трымаць комм хэша без зменаў, як я магу гэта зрабіць?

120

7 адказы

проста націснуць яшчэ якая ажыццяўляе на філіялу запыт для. Запыт цягнуць будзе забраць гэта тое

прыклад:

Калі вы хочаце мець б аб'яднаны ў майстар

  1. Націскаеш c1, c2, c3 б
  2. , то вы зробіце новы запыт на б
  3. ён атрымлівае агляд і вам трэба больш фіксацый
  4. Націскаеш c11, c21, c31 б
  5. Запыт цягнуць зараз паказвае ўсе 6 шэсць фіксацый
125
дададзена
Ёсць магчымасць дадаць фіксацыі на гэтую просьбу цягавага хто-то іншым, акрамя аўтара гэтага запыту цягнуць?
дададзена аўтар skywinder, крыніца
усе, хто штурхае да канкрэтнай галіны
дададзена аўтар Daij-Djan, крыніца
@downvoter: цалкам нармалёва, але чаму?
дададзена аўтар Daij-Djan, крыніца

Я толькі што быў адзін здзейсніць ў запыце цягнуць, і я выкарыстаў GIT здзейсніць --amend , каб абнавіць яго. Затым я рабіў намаганні штуршок з мярзотнік націскной -f так што мой Змененая комм замяніў арыгінал. Запыт цягнуць аўтаматычна падабраў новы комм. (Гэта на самай справе паказаў, як здзяйсняе, але калі я перазарадзіў старонку старога здзейсніць ўжо не было.)

Такім чынам, у той час як вымушаны штуршок, як правіла, не рэкамендуецца, гэта можа быць карысна для высоўных запытаў. Гэта не рэкамендуецца, таму што калі хто-то засноўвае фіксацыі на верхняй часткі твайго, то яны павінны будуць рабіць перабазавання пасля змены. Але так як ніхто не павінен засноўваць сваю працу на запыце цягавага пад разглядвалым, гэта павінна быць даволі бяспечна ў гэтай сітуацыі.

21
дададзена

Я толькі што быў адзін здзейсніць ў запыце цягнуць, і я выкарыстаў GIT здзейсніць --amend , каб абнавіць яго. Затым я рабіў намаганні штуршок з мярзотнік націскной -f так што мой Змененая комм замяніў арыгінал. Запыт цягнуць аўтаматычна падабраў новы комм. (Гэта на самай справе паказаў, як здзяйсняе, але калі я перазарадзіў старонку старога здзейсніць ўжо не было.)

Такім чынам, у той час як вымушаны штуршок, як правіла, не рэкамендуецца, гэта можа быць карысна для высоўных запытаў. Гэта не рэкамендуецца, таму што калі хто-то засноўвае фіксацыі на верхняй часткі твайго, то яны павінны будуць рабіць перабазавання пасля змены. Але так як ніхто не павінен засноўваць сваю працу на запыце цягавага пад разглядвалым, гэта павінна быць даволі бяспечна ў гэтай сітуацыі.

21
дададзена

Калі вы будзеце працягваць ўносіць змяненні і працягваць настойваць на тую ж галінку, вытанчаныя камітаў будуць дададзеныя ў той жа запыт цягнуць (калі ваш запыт цягнуць не зьлітыя). Гэта можа зрабіць гісторыю вельмі хаатычным.

Альтэрнатыўнае рашэнне, і метад, які я выкарыстоўваю выглядае наступным чынам:

  1. Create a new branch (fixes) from the repository(upstream) and branch (develop) to which you intend to send the pull request by doing:

    git branch fixes upstream/develop

  2. Add your refined commits directly to this newly created branch.

    git commit -m "your message"

  3. Push this branch to your own forked remote (could be named origin).

  4. Compare and send in a new pull request with clean commit history.
  5. Also, it is a good idea to delete your branch after the pull request has been merged.
  6. And you can comment and close your earlier pull requests.
6
дададзена

Калі вы будзеце працягваць ўносіць змяненні і працягваць настойваць на тую ж галінку, вытанчаныя камітаў будуць дададзеныя ў той жа запыт цягнуць (калі ваш запыт цягнуць не зьлітыя). Гэта можа зрабіць гісторыю вельмі хаатычным.

Альтэрнатыўнае рашэнне, і метад, які я выкарыстоўваю выглядае наступным чынам:

  1. Create a new branch (fixes) from the repository(upstream) and branch (develop) to which you intend to send the pull request by doing:

    git branch fixes upstream/develop

  2. Add your refined commits directly to this newly created branch.

    git commit -m "your message"

  3. Push this branch to your own forked remote (could be named origin).

  4. Compare and send in a new pull request with clean commit history.
  5. Also, it is a good idea to delete your branch after the pull request has been merged.
  6. And you can comment and close your earlier pull requests.
6
дададзена

Вы можаце таксама выкарыстоўваць GitHub API .

example with curl

curl --user "your_github_username" \
     --request PATCH \
     --data '{"title":"newtitle","body":"newbody",...}' \
     https://api.github.com/repos/:owner/:repo/pulls/:number

you can find the detailled list of data in github developer doc

example : change name of my pull request

curl --user "jeremyclement" \
     --request PATCH \
     --data '{"title":"allows the control of files and folders permissions."}' \
     https://api.github.com/repos/Gregwar/Cache/pulls/9
1
дададзена
Я спрабаваў змены галіны , але я думаю, галава/пазнака і галава/вых не рэдагуюцца.
дададзена аўтар Jer42, крыніца

Вы можаце таксама выкарыстоўваць GitHub API .

example with curl

curl --user "your_github_username" \
     --request PATCH \
     --data '{"title":"newtitle","body":"newbody",...}' \
     https://api.github.com/repos/:owner/:repo/pulls/:number

you can find the detailled list of data in github developer doc

example : change name of my pull request

curl --user "jeremyclement" \
     --request PATCH \
     --data '{"title":"allows the control of files and folders permissions."}' \
     https://api.github.com/repos/Gregwar/Cache/pulls/9
1
дададзена
Я спрабаваў змены галіны , але я думаю, галава/пазнака і галава/вых не рэдагуюцца.
дададзена аўтар Jer42, крыніца