Added __str__() to OrgDate #44
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When converting OrgDate (or its descendants) to a string, use the correct org format (considering active status, ranges)
|
thanks (and especially for adding a test :) ) I guess my only worry is that I'm not sure if generating org-mode ultimately should be the responsibility of orgparse (it's pretty tricky and could get out of hand). But I guess I don't mind for now if it's all stable and covered with tests :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added support to convert OrgDate and its descendants into strings, using the default org formats
activeflag<2021-09-03 Fri 16:01--17:02>by default if on the same day, but not forOrgClockDate)This pull request will probably conflict with #43, as I have put the necessary tests into the same new file, but I wanted to keep the PRs separate. Solving the merge conflict should be straightforward though!