Skip to content

Add support for merging codeprints.json files - #4

Open
mre wants to merge 52 commits into
masterfrom
merge
Open

Add support for merging codeprints.json files#4
mre wants to merge 52 commits into
masterfrom
merge

Conversation

@mre

@mre mre commented Feb 18, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

TODO:

  • Implement merge functionality
  • Add more tests
  • Make codeprints.json filenames unique (e.g. timestamp + repo name OR path OR unique hash)
@mre mre changed the title Add support for merging codeprints.json files Feb 18, 2021
@mre
mre marked this pull request as draft February 18, 2021 14:15
mre
mre commented Mar 10, 2021
View reviewed changes
Comment thread src/merge.rs Outdated
@mre
mre marked this pull request as ready for review March 10, 2021 00:32
@mre

mre commented Mar 10, 2021

Copy link
Copy Markdown
Contributor Author

This should be good to go now.
mre added 4 commits March 11, 2021 22:04
@mre @woolfg
Fix import
2a553a2
@mre @woolfg
Add subcommand for merge
250b815
@mre @woolfg
impl Default for Year (generating empty timelines)
27adefe
@mre @woolfg
Split up parser into ParseState and core Parser
42ca666 
This decouples the data representation from the parsing
process, making the internal data format usable for other
use-cases, such as merging timelines
@mre mre changed the title WIP: Add support for merging codeprints.json files Mar 15, 2021
@mre mre mentioned this pull request Mar 18, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@mre @woolfg