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

parse-buildlog.py でエラーの修正のために colors.YELLOW および colors.BLUE を使わない #1332

Merged

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Jul 4, 2020

PR の目的

カテゴリ

  • ビルド関連
    • ビルド手順
    • Azure Pipelines

PR の背景

現状で master で azure pipelines のビルドに失敗している。

原因は default の python が 2.7 → 3.7 に変更になったため
#1331 (comment)

PR のメリット

ビルドエラーになるのが修正される

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

(制限事項) ビルドエラーは修正されるが、生成される EXCEL ファイルの中身でヘッダ部分しか生成されない

仕様・動作説明

テスト内容

テスト1

手順

PR の影響範囲

関連 issue, PR

参考資料

@m-tmatma m-tmatma changed the title [WIP] parse-buildlog.py でエラーの修正のために colors.YELLOW および colors.BLUE を使わない parse-buildlog.py でエラーの修正のために colors.YELLOW および colors.BLUE を使わない Jul 4, 2020
@AppVeyorBot
Copy link

Build sakura 1.0.2870 completed (commit 8a39d66b18 by @m-tmatma)

@m-tmatma
Copy link
Member Author

m-tmatma commented Jul 4, 2020

#1333 の問題のためにこの PR のみでは azure pipelines のビルドエラーはすべては解消しません。

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.

一部問題が解消されたようですね。問題無いと思います。

@beru beru added appveyor CI appveyor など CI 関連 【ChangeLog除外】 azure pipelines and removed appveyor labels Jul 4, 2020
@m-tmatma m-tmatma merged commit 6714575 into sakura-editor:master Jul 4, 2020
@m-tmatma m-tmatma deleted the feature/not-to-use-some-handy-colors branch July 11, 2020 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
azure pipelines CI appveyor など CI 関連 【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants