2020-02-14 07:50:38 +00:00
|
|
|
// PR commentary for Authelia branch based contributions
|
|
|
|
on('pull_request.opened')
|
|
|
|
.filter(
|
|
|
|
context =>
|
2020-07-01 06:07:27 +00:00
|
|
|
context.payload.pull_request.head.label.startsWith('authelia:')
|
2020-02-14 07:50:38 +00:00
|
|
|
)
|
2020-03-15 02:49:57 +00:00
|
|
|
.filter(
|
|
|
|
context =>
|
2020-07-01 06:07:27 +00:00
|
|
|
!context.payload.pull_request.head.ref.startsWith('dependabot/')
|
2020-03-15 02:49:57 +00:00
|
|
|
)
|
[FEATURE] Notifier Startup Checks (#889)
* implement SMTP notifier startup check
* check dial, starttls, auth, mail from, rcpt to, reset, and quit
* log the error on failure
* implement mock
* misc optimizations, adjustments, and refactoring
* implement validate_skip config option
* fix comments to end with period
* fix suites that used smtp notifier without a smtp container
* add docs
* add file notifier startup check
* move file mode into const.go
* disable gosec linting on insecureskipverify since it's intended, warned, and discouraged
* minor PR commentary adjustment
* apply suggestions from code review
Co-Authored-By: Amir Zarrinkafsh <nightah@me.com>
2020-04-21 04:59:38 +00:00
|
|
|
.comment(`## Artifacts
|
2020-04-21 03:08:40 +00:00
|
|
|
These changes are published for testing on Buildkite and DockerHub.
|
|
|
|
|
[FEATURE] Notifier Startup Checks (#889)
* implement SMTP notifier startup check
* check dial, starttls, auth, mail from, rcpt to, reset, and quit
* log the error on failure
* implement mock
* misc optimizations, adjustments, and refactoring
* implement validate_skip config option
* fix comments to end with period
* fix suites that used smtp notifier without a smtp container
* add docs
* add file notifier startup check
* move file mode into const.go
* disable gosec linting on insecureskipverify since it's intended, warned, and discouraged
* minor PR commentary adjustment
* apply suggestions from code review
Co-Authored-By: Amir Zarrinkafsh <nightah@me.com>
2020-04-21 04:59:38 +00:00
|
|
|
### Docker Container
|
2020-02-14 07:50:38 +00:00
|
|
|
* \`docker pull authelia/authelia:{{ pull_request.head.ref }}\``)
|
|
|
|
|
|
|
|
// PR commentary for third party based contributions
|
|
|
|
on('pull_request.opened')
|
|
|
|
.filter(
|
|
|
|
context =>
|
2020-07-01 06:07:27 +00:00
|
|
|
context.payload.pull_request.head.label.startsWith('authelia:')
|
|
|
|
)
|
|
|
|
.filter(
|
|
|
|
context =>
|
|
|
|
!context.payload.pull_request.head.label.endsWith(':master')
|
2020-02-14 07:50:38 +00:00
|
|
|
)
|
2020-06-17 06:39:18 +00:00
|
|
|
.comment(`Thanks for choosing to contribute. We lint all PR's with golangci-lint, I may add a review to your PR with some suggestions.
|
[FEATURE] Notifier Startup Checks (#889)
* implement SMTP notifier startup check
* check dial, starttls, auth, mail from, rcpt to, reset, and quit
* log the error on failure
* implement mock
* misc optimizations, adjustments, and refactoring
* implement validate_skip config option
* fix comments to end with period
* fix suites that used smtp notifier without a smtp container
* add docs
* add file notifier startup check
* move file mode into const.go
* disable gosec linting on insecureskipverify since it's intended, warned, and discouraged
* minor PR commentary adjustment
* apply suggestions from code review
Co-Authored-By: Amir Zarrinkafsh <nightah@me.com>
2020-04-21 04:59:38 +00:00
|
|
|
|
2020-04-21 03:08:40 +00:00
|
|
|
You are free to apply the changes if you're comfortable, alternatively you are welcome to ask a team member for advice.
|
|
|
|
|
[FEATURE] Notifier Startup Checks (#889)
* implement SMTP notifier startup check
* check dial, starttls, auth, mail from, rcpt to, reset, and quit
* log the error on failure
* implement mock
* misc optimizations, adjustments, and refactoring
* implement validate_skip config option
* fix comments to end with period
* fix suites that used smtp notifier without a smtp container
* add docs
* add file notifier startup check
* move file mode into const.go
* disable gosec linting on insecureskipverify since it's intended, warned, and discouraged
* minor PR commentary adjustment
* apply suggestions from code review
Co-Authored-By: Amir Zarrinkafsh <nightah@me.com>
2020-04-21 04:59:38 +00:00
|
|
|
## Artifacts
|
2020-04-21 03:08:40 +00:00
|
|
|
These changes once approved by a team member will be published for testing on Buildkite and DockerHub.
|
|
|
|
|
[FEATURE] Notifier Startup Checks (#889)
* implement SMTP notifier startup check
* check dial, starttls, auth, mail from, rcpt to, reset, and quit
* log the error on failure
* implement mock
* misc optimizations, adjustments, and refactoring
* implement validate_skip config option
* fix comments to end with period
* fix suites that used smtp notifier without a smtp container
* add docs
* add file notifier startup check
* move file mode into const.go
* disable gosec linting on insecureskipverify since it's intended, warned, and discouraged
* minor PR commentary adjustment
* apply suggestions from code review
Co-Authored-By: Amir Zarrinkafsh <nightah@me.com>
2020-04-21 04:59:38 +00:00
|
|
|
### Docker Container
|
2020-07-01 06:07:27 +00:00
|
|
|
* \`docker pull authelia/authelia:PR{{ pull_request.number }}\``)
|
|
|
|
|
|
|
|
// PR commentary for forked master branches
|
|
|
|
on('pull_request.opened')
|
|
|
|
.filter(
|
|
|
|
context =>
|
|
|
|
context.payload.pull_request.head.label.startsWith('authelia:')
|
|
|
|
)
|
|
|
|
.filter(
|
|
|
|
context =>
|
|
|
|
context.payload.pull_request.head.label.endsWith(':master')
|
|
|
|
)
|
|
|
|
.comment(`Thanks for choosing to contribute. It appears that you're submitting a PR from a forked master branch.
|
|
|
|
|
|
|
|
This causes issues with codecov, please close this PR and re-submit your PR from a branch other than master.`)
|