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.

Sign up for GitHub

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

Jump to bottom

Write version of DebugWriter #65

Closed
jacobthill opened this issue Feb 22, 2019 · 1 comment · Fixed by #66
Closed

Write version of DebugWriter #65

jacobthill opened this issue Feb 22, 2019 · 1 comment · Fixed by #66
Projects
DLME Maintenance Work Cycle - Early 2019

Comments

@jacobthill
Copy link
Contributor

We need to write our own version of DebugWriter. There is an error in LineWriter traject/traject#202 that is causing problems when trying to run on a directory of files. I use DebugWriter to check that the transformations I'm making are doing what they are supposed to. I understand that DebugWriter will not check the IR. I will use JsonWriter for that once I'm happy with the transformations.
@justinlittman
Copy link
Contributor

Also, would be helpful to have DebugWriter as a commandline flag so that don't need to change writer in config.

justinlittman added a commit that referenced this issue Feb 22, 2019
@justinlittman
Adds patched debug writer. Adds debug writer as commandline option.
d49f2d1 
closes #65
@justinlittman justinlittman mentioned this issue Feb 22, 2019
Merged
justinlittman added a commit that referenced this issue Feb 22, 2019
@justinlittman
Adds patched debug writer. Adds debug writer as commandline option.
6e329a5 
closes #65
justinlittman added a commit that referenced this issue Feb 25, 2019
@justinlittman
Adds patched debug writer. Adds debug writer as commandline option.
88e3ab8 
closes #65
justinlittman added a commit that referenced this issue Feb 25, 2019
@justinlittman
Adds patched debug writer. Adds debug writer as commandline option.
e8ecf00 
closes #65
DLME Maintenance Work Cycle - Early 2019 automation moved this from In Progress to Done Feb 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
No open projects
DLME Maintenance Work Cycle - Early 2019
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds patched debug writer. Adds debug writer as commandline option. sul-dlss/dlme-transform
2 participants
@justinlittman @jacobthill