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:56
@derelbenkoenig derelbenkoenig requested review from luckysoni and removed request for a team September 23, 2025 20:56
@derelbenkoenig derelbenkoenig enabled auto-merge (squash) September 23, 2025 21:06
@luckysoni luckysoni requested review from a team and OlaoluwaM and removed request for a team and luckysoni September 24, 2025 07:43
@derelbenkoenig derelbenkoenig merged commit 070cc26 into main Oct 31, 2025
9 checks passed
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.

3 participants