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.

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

Feature: Slack notifications #6

Open
pawroman opened this issue Jan 29, 2021 · 1 comment
Open

Feature: Slack notifications #6

pawroman opened this issue Jan 29, 2021 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@pawroman
Copy link
Member

As a feature, we could support Slack notifications when link checking fails.

This could be done as a webhook or alternatively, using ready-made GitHub actions or other integrations. In this case the README could include example of how to enable this integration.

@mre mre added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Feb 12, 2021
@mre
Copy link
Member

mre commented Aug 23, 2021

If someone wants to give this a try, there are Github actions like these that we could use: https://github.com/marketplace/actions/slack-notify. I'd prefer that over bundling the Slack integration into lychee-action, because it would be more flexible to configure and it would be easier to stay up-to-date with Slack API changes.

Steps to help out here:

  1. Add lychee and Slack action to a test pipeline
  2. Try a test run to see if it works (e.g. it sends a Slack message on lychee failure)
  3. Send a pull request which adds a usage example to the README.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants