Skip to content

Conversation

@derelbenkoenig
Copy link
Contributor

Summary

This PR moves markdown-unlit from dependencies to build-tools in the package.yaml test configuration and removes the corresponding empty import from README.lhs.

Why this change?

When markdown-unlit is listed in dependencies, we need to import it in the Haskell code to avoid unused-package warnings from GHC. However, since markdown-unlit is only used as a preprocessor (via -pgmL markdown-unlit), we don't actually need to import anything from it in our code.

By moving it to build-tools instead, we:

  • Avoid unused-package warnings without needing empty imports
  • Make the dependency relationship clearer (it's a build tool, not a runtime dependency)
  • Follow Stack/Cabal best practices for preprocessor dependencies

Changes

  • Moved markdown-unlit from dependencies to build-tools in package.yaml
  • Removed import Text.Markdown.Unlit () from README.lhs
  • Regenerated any files via stack build --fast --test --no-run-tests

🤖 Generated with Claude Code

This avoids unused-package warnings and removes the need for empty imports.

🤖 Generated with Claude Code
@derelbenkoenig derelbenkoenig requested a review from a team as a code owner September 23, 2025 20:35
@derelbenkoenig derelbenkoenig requested review from OlaoluwaM and removed request for a team September 23, 2025 20:35
@derelbenkoenig
Copy link
Contributor Author

so close but it destroyed the whitespace in the yaml, trying again

@derelbenkoenig derelbenkoenig deleted the markdown-unlit-build-tools-20250923-163523 branch September 23, 2025 20:44
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