search

RockefellerArchiveCenter / scripts

Useful scripts
MIT License
16 stars 5 forks source link

Add print notes script and make level argument optional #76

Closed kemartin2015 closed 3 years ago

kemartin2015 commented 3 years ago

Hi @helrond, when you get a chance, can you look at the change to the logic I made in line 31 of the edit notes script to make the level argument optional? I made the same change to the print_notes.py script. Thanks!

kemartin2015 commented 3 years ago

This makes sense, @helrond. Your suggestions would make it much easier for someone reading the code for the first time. I'll play around with both options. Thank you!

kemartin2015 commented 3 years ago

@helrond I think for the sake of future projects, I like having to specify the level when running the script. I made the level argument required and added all as one of the choices. Do my changes look okay?