Added output CruiseControl file feature#36
Open
hidenorigoto wants to merge 1 commit intopiece:masterfrom
Open
Conversation
Member
|
ありがとうございます。 本機能のユースケースは、ローカルまたはリモートの CI サーバーによるテスト実行結果のローカルマシンへの通知と考えてよろしいでしょうか。そのようなユースケースでしたら、既存の通知と併用するような需要はなさそうに思いますので、通知機能への統合は妥当だと思います。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added a feature which enables output test status to CruiseControl XML file by
--cc-fileoption (specifing file path), and--cc-nameoption (specifing a project name embeded to XML file).In the current implemenataion, 2 options above (
--cc-file,--cc-name) are exclusive to--notify, because in my opinion, they are exclusive usecases.CruiseControlのXMLフォーマットでテストステータスを出力する機能を追加しました。
--cc-fileで出力先ファイルのパスを指定すると、テスト終了時に結果を出力します。--cc-nameオプションで、出力するファイル内のプロジェクト名を指定できます。この機能は、既存の通知 (
--notify) の拡張としているので、現在の実装では上記2つのオプションと--notifyとは排他的な関係になっています。Configurationの段階で共通のnotifyエントリを使っています。ただ、両方同時に使える方が一般的かもしれません。この場合はConfigurationを独立させるように変更します。
ご意見お願いします。
参考: