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

Add support for missing CLI options #25

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

DeutscheGabanna
Copy link
Owner

@DeutscheGabanna DeutscheGabanna commented Aug 4, 2024

Program now (hopefully) correctly responds to options that have been part of the help, but did not have any implementation:

  • prefix
  • suffix
  • auto-reject all file conflicts
  • auto-accept all file overwrites

Contents (#25)

New features

  • Auto-accept or auto-refuse writing over existing files in case of conflict

Enhancements

  • Filter out duplicate logs

Fixes

  • Use custom prefix and suffix in entry headings if set
  • typo in Librarian class
  • Adjust expected files so that they reflect actual proper output

Refactoring

  • Handle .md files in a dedicated class

Copy link

github-actions bot commented Aug 4, 2024

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/daylio_to_md
  errors.py 41-47
  group.py 140
  journal_entry.py
  librarian.py 209-245, 251, 293
  utils.py 326
  src/daylio_to_md/entry
  mood.py
  tests
  test_journal_entry.py
  test_output.py
Project Total  

This report was generated by python-coverage-comment-action

@DeutscheGabanna DeutscheGabanna force-pushed the all_settings_should_do_something branch 2 times, most recently from c61ffa6 to eceb598 Compare August 4, 2024 13:21
@DeutscheGabanna DeutscheGabanna force-pushed the all_settings_should_do_something branch from eceb598 to 0727f4c Compare August 4, 2024 13:23
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.

1 participant