Skip to content

Add support for missing CLI options#25

Open
DeutscheGabanna wants to merge 6 commits intomainfrom
all_settings_should_do_something
Open

Add support for missing CLI options#25
DeutscheGabanna wants to merge 6 commits intomainfrom
all_settings_should_do_something

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

@github-actions
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