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

batch ファイルのキーワードに setlocal/endlocal を追加 #1171

Merged

Conversation

m-tmatma
Copy link
Member

PR の目的

batch ファイルのキーワードに setlocal/endlocal を追加

カテゴリ

  • 機能追加

PR の背景

batch ファイルのキーワードで setlocal/endlocal が入っていなかったので追加する

PR のメリット

setlocal/endlocal がキーワードとして認識される。

PR のデメリット (トレードオフとかあれば)

既存の sakura.ini は変更しないので新規インストールのみ有効

PR の影響範囲

batch ファイルのキーワード表示

関連チケット

参考資料

@m-tmatma m-tmatma added this to the v2.4.0 milestone Jan 25, 2020
@AppVeyorBot
Copy link

Build sakura 1.0.2564 failed (commit 0d597f200e by @m-tmatma)

@berryzplus
Copy link
Contributor

アフォになった感じで何も考えずに「OKっす!」といっとくか、
「そもそもキーワードとは何だろう?」とか哲学的考察を開始するか。

・・・感覚でやって構わないと思うんですよね。

そんなことより、v2.4.0 タグを付けてるのが気になってます。

マージ前に v2.4.0 タグを付ける行為は、
この変更が現状のリリース作業に待ったをかける重要な変更である、
という意思表示と受け取れます。

そうなんだっけ?
というのが引っかかりポイントです。

@berryzplus
Copy link
Contributor

ビルド失敗していますが、問題ないと看做してよいと思います。

checking https://ci.appveyor.com/projects/sakuraeditor/sakura/builds/30348718
caught an error.
Invoke-RestMethod : No such host is known
At C:\projects\sakura\help\extract-chm-from-artifact.ps1:22 char:13

  • ... $project = Invoke-RestMethod -Method Get -Uri "$apiUrl/projects/$acc ...
  •             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    
  • CategoryInfo : InvalidOperation: (Method: GET, Reques\u2026PowerShell/6.2.3
    }:HttpRequestMessage) [Invoke-RestMethod], HttpRequestException
  • FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand
    error build-chm.bat

ビルド済みヘルプファイルをappveyorからダウンロードしようとして失敗している。
このPRの変更対象のビルドが終わったあとの処理でコケた感じなのでスルーして問題ないっす。

@m-tmatma
Copy link
Member Author

そんなことより、v2.4.0 タグを付けてるのが気になってます。

2.4.0 に入らなかったら、外せばいいだけです。
後でどのリリースに何が入っているかわかるようにするためのものです。

この変更が現状のリリース作業に待ったをかける重要な変更である、

重要な変更に関しては、IMPORTANT のラベルをつける、あるいはリリース関連の
チケットにそのことを記載するなどの運用になっているかと思います。

@AppVeyorBot
Copy link

Build sakura 1.0.2566 completed (commit 0d597f200e by @m-tmatma)

@berryzplus
Copy link
Contributor

そんなことより、v2.4.0 タグを付けてるのが気になってます。

2.4.0 に入らなかったら、外せばいいだけです。
後でどのリリースに何が入っているかわかるようにするためのものです。

了解。

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

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

問題ないと思います。

@m-tmatma m-tmatma merged commit ed4df73 into sakura-editor:master Jan 25, 2020
@KENCHjp KENCHjp added the enhancement ■機能追加 label Feb 4, 2020
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
…rd-setlocal-endlocal

batch ファイルのキーワードに setlocal/endlocal を追加
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ■機能追加
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants