Skip to content

Add configurable file suffix for generated markdown files#40

Merged
liran-funaro merged 2 commits intoliran-funaro:mainfrom
jdillard:feature/out-suffix
Oct 15, 2025
Merged

Add configurable file suffix for generated markdown files#40
liran-funaro merged 2 commits intoliran-funaro:mainfrom
jdillard:feature/out-suffix

Conversation

@jdillard
Copy link

@jdillard jdillard commented Oct 14, 2025

Motivation

This PR adds support for customizing the file extension of generated markdown files to support conventions like https://llmstxt.org/, which suggests using the .html.md naming pattern (e.g., index.html.md).

Previously, the output file suffix was hardcoded to .md, which meant users couldn't generate files with alternative extensions

Summary of Changes

  • Added markdown_file_suffix config value (default: .md)
  • Created dedicated test_custom_file_suffix() to verify files are generated with correct extension
  • Updated _chmod_output() helper to handle both .md and .html.md files

Relates to #36

@coveralls
Copy link

Pull Request Test Coverage Report for Build 18484897391

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.003%) to 99.103%

Totals Coverage Status
Change from base Build 13610142503: 0.003%
Covered Lines: 773
Relevant Lines: 780

💛 - Coveralls

Copy link

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

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

This is really helpful thanks!

Copy link
Owner

@liran-funaro liran-funaro left a comment

Choose a reason for hiding this comment

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

LGTM

@liran-funaro liran-funaro merged commit 39e4d66 into liran-funaro:main Oct 15, 2025
6 of 7 checks passed
@jdillard jdillard deleted the feature/out-suffix branch October 15, 2025 18:05
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.

4 participants