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.

Sign up for GitHub

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

Jump to bottom

Notify Travis Status should handle builds triggered by API #8025

Closed
pomek opened this issue Sep 4, 2020 · 1 comment
Closed

Notify Travis Status should handle builds triggered by API #8025

pomek opened this issue Sep 4, 2020 · 1 comment
Assignees
@pomek
Labels
package:dev squad:platform Issue to be handled by the Platform team. type:improvement This issue reports a possible enhancement of an existing feature.
Milestone
iteration 36 (v23...

Comments

@pomek
Copy link
Member

pomek commented Sep 4, 2020

The script that sends a notification to Slack should work also with builds triggered via API.

https://github.com/ckeditor/ckeditor5-dev/blob/8c61cc519a89eca9c604f3613fbc42dbd946517b/packages/ckeditor5-dev-tests/bin/notify-travis-status.js#L28-L31
@pomek pomek added type:improvement This issue reports a possible enhancement of an existing feature. squad:platform Issue to be handled by the Platform team. package:dev labels Sep 4, 2020
@pomek pomek added this to the iteration 36 milestone Sep 4, 2020
@pomek pomek self-assigned this Sep 4, 2020
@pomek
Copy link
Member Author

pomek commented Sep 4, 2020

Fixed in ckeditor/ckeditor5-dev@7c05326.

@pomek pomek closed this as completed Sep 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pomek pomek

Labels
package:dev squad:platform Issue to be handled by the Platform team. type:improvement This issue reports a possible enhancement of an existing feature.
Projects
None yet
Milestone
iteration 36 (v23.0.0)
Development

No branches or pull requests
1 participant
@pomek