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

direct message feature branch build failures #81

Open
ygrek opened this issue Aug 19, 2020 · 0 comments
Open

direct message feature branch build failures #81

ygrek opened this issue Aug 19, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@ygrek
Copy link
Contributor

ygrek commented Aug 19, 2020

Enable CI notifications only for main branch, for branches - send CI status as direct message to the author of commit on branch
Configurable on/off per-person (easy: via config file, future: via chatting with bot)
Duplicate all direct message to default channel for observability anyway
@ygrek ygrek added the enhancement New feature or request label Aug 19, 2020
ygrek pushed a commit that referenced this issue Oct 23, 2020
@rr0gi
non-main branch build notifications go to default channel only (ref #81)
4bf48c8
yasunariw added a commit that referenced this issue Dec 8, 2020
@yasunariw
Merge branch 'master' into feature/#43
a35a09b 
* master: (39 commits)
  Add devkit.core to test
  Add devkit.core to src
  re-add devkit.core dependency removed in 07d3899
  tests: update expected with change from 4bf48c8
  tests: make tests compile again with explicit transitive deps
  implicit_transitive_deps false
  non-main branch build notifications go to default channel only (ref #81)
  minor
  tests: promote
  highlight author name
  slack: fix unescaping of parentheses
  slack: tweak escaping
  slack: properly transform img links
  issues: show body only for "new issue" notification
  better branch list in CI notification (fix #73)
  minor improvement for CI notification message
  filter out merges of main branch into feature after feature branch is merged into main
  better logs
  use standard json
  specify success state as tristate switch
  ...
yasunariw pushed a commit that referenced this issue Nov 30, 2021
@rr0gi
non-main branch build notifications go to default channel only (ref #81)
2edb227
yasunariw added a commit that referenced this issue Nov 30, 2021
@yasunariw
Merge branch 'master' into feature/#43
f3c0f35 
* master: (39 commits)
  Add devkit.core to test
  Add devkit.core to src
  re-add devkit.core dependency removed in 4137e70
  tests: update expected with change from 2edb227
  tests: make tests compile again with explicit transitive deps
  implicit_transitive_deps false
  non-main branch build notifications go to default channel only (ref #81)
  minor
  tests: promote
  highlight author name
  slack: fix unescaping of parentheses
  slack: tweak escaping
  slack: properly transform img links
  issues: show body only for "new issue" notification
  better branch list in CI notification (fix #73)
  minor improvement for CI notification message
  filter out merges of main branch into feature after feature branch is merged into main
  better logs
  use standard json
  specify success state as tristate switch
  ...
@koonwen koonwen mentioned this issue Aug 8, 2023
Merged
@Khady Khady mentioned this issue Feb 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ygrek