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

GitHub actions email on failure #140

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

manicki
Copy link
Member

@manicki manicki commented Sep 16, 2024

Further work to move over from Travis CI.
Includes the commit from #138

Relevant commit for emails on failure is dc3b2b9 and following ones

@manicki
Copy link
Member Author

manicki commented Sep 16, 2024

meh, think I shall try defining the email content in markdown instead of writing HTML in yaml file

@manicki
Copy link
Member Author

manicki commented Sep 19, 2024

substituting github actions context variables in a file (does not matter if markdown or HTML) does not currently work, so gotta stick to writing HTML, or maybe markdown is slightly better, in yaml file for now.

@manicki
Copy link
Member Author

manicki commented Sep 20, 2024

Figure out how to use Github action context variable in external files defining the content of CI failure emails. This PR branch will need a rebase after the "parent" PR is potentially merged any way, so keeping the markdown as a separate commit until then. It makes the env variable substitution a bit more visible and I would actually like to hear about this the most.

@manicki manicki force-pushed the github-actions-email-on-failure branch from de22671 to 9b6dd9a Compare September 20, 2024 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant