Skip to content

Conversation

Ma27
Copy link

@Ma27 Ma27 commented Aug 24, 2025

Purpose

That allows downstream packages to e.g. split the builds of frontend/backend/mail, as it's the case in nixpkgs: NixOS/nixpkgs@5597a4f

Proposal

  • item 1...
  • item 2...

External contributions

Thank you for your contribution! 🎉

Please ensure the following items are checked before submitting your pull request:

  • I have read and followed the contributing guidelines
  • I have read and agreed to the Code of Conduct
  • I have signed off my commits with git commit --signoff (DCO compliance)
  • I have signed my commits with my SSH or GPG key (git commit -S)
  • My commit messages follow the required format: <gitmoji>(type) title description
  • I have added a changelog entry under ## [Unreleased] section (if noticeable change)
  • I have added corresponding tests for new features or bug fixes (if applicable)

That allows downstream packages to e.g. split the builds of
frontend/backend/mail, as it's the case in nixpkgs:
NixOS/nixpkgs@5597a4f

Signed-off-by: Maximilian Bosch <[email protected]>
@@ -1,7 +1,7 @@
#!/usr/bin/env bash
set -eo pipefail
# Run html-to-text to convert all html files to text files
DIR_MAILS="../backend/core/templates/mail/"
DIR_MAILS="${DIR_MAILS:-../backend/core/templates/mail}/"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea but maybe use a prefix ? DOCS_DIR_MAILS ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants