pcdshub / pre-commit-hooks

Pre-commit hooks for PCDS projects (https://pre-commit.com/)
Other
10 stars 7 forks source link

add new lines to the start and end of each structured text section #23

Closed aeu485 closed 6 months ago

aeu485 commented 12 months ago

Addresses #10. Works upon ST declaration and implementations, with the --lines argument for specifying additional \n. Two modifications occur from the use ElementTree.Write:

ZLLentz commented 7 months ago

This is well implemented and a good addition, thank you! I'm stalling out a bit on review due to some busy season/staffing shortages, it will take me a bit longer to verify the versioneer update (I know it's stock and standard, but it's also a big diff and I can't accept it without reviewing it thoroughly)

aeu485 commented 7 months ago

No problem. I removed the versioneer updates as you didn't update in your most recent PR either.

ZLLentz commented 6 months ago

merging and then I'm going to make a mini follow-up PR to rename the file so it will run from the pre-commit binary

thank you for your contribution and your patience!