2020-02-14 14:50:38 +07:00
|
|
|
// PR commentary for Authelia branch based contributions
|
|
|
|
on('pull_request.opened')
|
|
|
|
.filter(
|
|
|
|
context =>
|
2020-07-01 13:28:12 +07:00
|
|
|
context.payload.pull_request.head.label.slice(0, 9) === 'authelia:'
|
2020-02-14 14:50:38 +07:00
|
|
|
)
|
2020-03-15 09:49:57 +07:00
|
|
|
.filter(
|
|
|
|
context =>
|
2020-07-01 13:28:12 +07:00
|
|
|
context.payload.pull_request.head.ref.slice(0, 11) !== 'dependabot/'
|
2020-03-15 09:49:57 +07: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 11:59:38 +07:00
|
|
|
.comment(`## Artifacts
|
2020-04-21 10:08:40 +07: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 11:59:38 +07:00
|
|
|
### Docker Container
|
2020-02-14 14:50:38 +07: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 13:28:12 +07:00
|
|
|
context.payload.pull_request.head.label.slice(0, 9) !== 'authelia:'
|
2020-07-01 13:07:27 +07:00
|
|
|
)
|
|
|
|
.filter(
|
|
|
|
context =>
|
2020-07-01 13:28:12 +07:00
|
|
|
!context.payload.pull_request.head.label.includes(':master')
|
2020-02-14 14:50:38 +07:00
|
|
|
)
|
2020-06-17 13:39:18 +07: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 11:59:38 +07:00
|
|
|
|
2020-04-21 10:08:40 +07: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 11:59:38 +07:00
|
|
|
## Artifacts
|
2020-04-21 10:08:40 +07: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 11:59:38 +07:00
|
|
|
### Docker Container
|
2020-07-01 13:07:27 +07:00
|
|
|
* \`docker pull authelia/authelia:PR{{ pull_request.number }}\``)
|
|
|
|
|
|
|
|
// PR commentary for forked master branches
|
|
|
|
on('pull_request.opened')
|
|
|
|
.filter(
|
|
|
|
context =>
|
2020-07-01 13:28:12 +07:00
|
|
|
context.payload.pull_request.head.label.slice(0, 9) !== 'authelia:'
|
2020-07-01 13:07:27 +07:00
|
|
|
)
|
|
|
|
.filter(
|
|
|
|
context =>
|
2020-07-01 13:28:12 +07:00
|
|
|
context.payload.pull_request.head.label.includes(':master')
|
2020-07-01 13:07:27 +07:00
|
|
|
)
|
|
|
|
.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.`)
|