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.

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

Pull Request を簡単にローカルに取得するためのバッチファイルを追加 (take2) #440

Merged
merged 13 commits into from
Sep 12, 2018

Conversation

m-tmatma
Copy link
Member

Pull Request を簡単にローカルに取得するためのバッチファイルを追加 (take2)

#38 で指摘された問題を解決した PR です。
PR がクローズ状態のまま、rebase して force update したので
再オープンできなくなったので新しい PR を作成しました。

ローカルブランチを作るときに ブランチ名に現在の日時を入れることで
取得後に更新された場合でも対応できるようにしました。

get-PR.bat Outdated Show resolved Hide resolved
get-PR.bat Outdated Show resolved Hide resolved
get-PR.md Show resolved Hide resolved
@m-tmatma m-tmatma added this to the next release milestone Sep 12, 2018
@m-tmatma
Copy link
Member Author

m-tmatma commented Sep 12, 2018

ちなみに markdown の表は

sakura-editor/management-forum#31 (comment) で書いた
以下ソフトを使いました。表を作るのが楽しくなります。

選択したExcelのセルをMarkdown形式でコピーするExcelアドインです。
http://www.nuits.jp/entry/copytomarkdownaddin

ソースは以下にあります。
https://github.com/nuitsjp/CopyToMarkdownAddIn (英語)

デモ動画もあります。
https://github.com/nuitsjp/CopyToMarkdownAddIn/blob/master/README-jp.md (日本語)

beru
beru previously approved these changes Sep 12, 2018
Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

問題無いと思います。
README.md の中で PR を Pull Request と書いた方が初めて読む人にとっては読み易いかもしれないと思いましたが、開発者ならきっとわかるでしょう。

@beru
Copy link
Contributor

beru commented Sep 12, 2018

しかし自分で動かさないで Merge してしまうのはまずいですね。。
今から取り込んで動かしてみます…。

@m-tmatma
Copy link
Member Author

README.md の中で PR を Pull Request と書いた方が初めて読む人にとっては読み易いかもしれないと思いましたが、開発者ならきっとわかるでしょう。

一応補足しました。

@beru
Copy link
Contributor

beru commented Sep 12, 2018

動かしてみました。

Microsoft Windows [Version 10.0.17134.228]
(c) 2018 Microsoft Corporation. All rights reserved.

C:\Users\haswell>d:

D:\>cd D:\projects\sakura

D:\projects\sakura>get-PR.bat
usage   : get-PR.bat PR-Number [remote name]
example1: get-PR.bat 1
example2: get-PR.bat 1 upstream
example3: get-PR.bat 1 origin

[remote name]
1. If "remote name" is valid, it is used as the origin of PR.
2. If "remote name" is invalid and env SAKURA_EDITOR_REMOTE_NAME is valid,
   env SAKURA_EDITOR_REMOTE_NAME is used as the origin of PR.
3. Otherwise, "origin" is used as the origin of PR.

D:\projects\sakura>get-PR.bat 441 github/sakura-editor
fetching pull request 441 from github/sakura-editor
git fetch github/sakura-editor pull/441/head:PR-441/20180913-074524
From https://github.com/sakura-editor/sakura
 * [new ref]             refs/pull/441/head -> PR-441/20180913-074524
checkout PR 441 to branch PR-441/20180913-074524
git checkout PR-441/20180913-074524
Switched to branch 'PR-441/20180913-074524'
バッチ ファイルが見つかりません。

D:\projects\sakura>get-PR.bat 999 github/sakura-editor
fetching pull request 999 from github/sakura-editor
git fetch github/sakura-editor pull/999/head:PR-999/20180913-075017
fatal: Couldn't find remote ref pull/999/head

D:\projects\sakura>get-PR.bat -999 github/sakura-editor
fetching pull request -999 from github/sakura-editor
git fetch github/sakura-editor pull/-999/head:PR--999/20180913-075027
fatal: Couldn't find remote ref pull/-999/head

D:\projects\sakura>get-PR.bat 0 github/sakura-editor
fetching pull request 0 from github/sakura-editor
git fetch github/sakura-editor pull/0/head:PR-0/20180913-075034
fatal: Couldn't find remote ref pull/0/head

D:\projects\sakura>get-PR.bat 440 github/sakura-editor
fetching pull request 440 from github/sakura-editor
git fetch github/sakura-editor pull/440/head:PR-440/20180913-075044
remote: Counting objects: 4, done.
remote: Compressing objects: 100% (1/1), done.
remote: Total 4 (delta 3), reused 4 (delta 3), pack-reused 0
Unpacking objects: 100% (4/4), done.
From https://github.com/sakura-editor/sakura
 * [new ref]             refs/pull/440/head -> PR-440/20180913-075044
checkout PR 440 to branch PR-440/20180913-075044
git checkout PR-440/20180913-075044
Switched to branch 'PR-440/20180913-075044'

D:\projects\sakura>

Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@beru
Copy link
Contributor

beru commented Sep 12, 2018

確認したところ問題無さそうなので Merge します。
もし問題が見つかった場合は別のPRで対応する事にしましょう。

@beru beru merged commit 749c669 into sakura-editor:master Sep 12, 2018
@m-tmatma m-tmatma deleted the feauture/easy-PR-review branch September 12, 2018 23:01
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…eview

Pull Request を簡単にローカルに取得するためのバッチファイルを追加 (take2)
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.

2 participants