Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow custom file extensions for directory journal #1873

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

comkieffer
Copy link

This PR builds on #1789 (Closes issue #1289).

In short, it adds a new configuration option extension that controls the extension for the files created with a folder journal.

Checklist

  • I have read the contributing doc.
  • I have included a link to the relevant issue number.
  • I have checked to ensure there aren't other open pull requests
    for the same issue.
  • I have written new tests for these changes, as needed.

@comkieffer comkieffer force-pushed the allow-custom-file-extensions-for-directory-journal branch 2 times, most recently from 68590e7 to 5b5a3c4 Compare March 23, 2024 19:22
The test previously only tested that one of the expected files was
created.
@comkieffer comkieffer force-pushed the allow-custom-file-extensions-for-directory-journal branch from 5b5a3c4 to 386c222 Compare March 23, 2024 19:43
@thw26
Copy link

thw26 commented Apr 3, 2024

Thanks for building on my PR! I have not had the time to return to it since the review.

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.

None yet

2 participants