Skip to content

Move markdown-unlit to build-tools#26

Merged
derelbenkoenig merged 2 commits intomainfrom
markdown-unlit-build-tools-20250923-172520
Sep 24, 2025
Merged

Move markdown-unlit to build-tools#26
derelbenkoenig merged 2 commits intomainfrom
markdown-unlit-build-tools-20250923-172520

Conversation

@derelbenkoenig
Copy link
Copy Markdown
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 21:25
@derelbenkoenig derelbenkoenig requested review from pbrisbin and removed request for a team September 23, 2025 21:25
@derelbenkoenig derelbenkoenig enabled auto-merge (squash) September 23, 2025 21:25
@derelbenkoenig derelbenkoenig merged commit ce4fe46 into main Sep 24, 2025
13 checks passed
@derelbenkoenig derelbenkoenig deleted the markdown-unlit-build-tools-20250923-172520 branch September 24, 2025 16:04
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