implemented feature: shell script relative paths #8

allymander commented 2026年03月05日 23:24:13 +01:00

Implemented feature request from #7. Implementation could use some cleanup for codebase aesthetics, but works properly on my machine. The logic permits storing executables in layout_dir and arbitrarily deep subdirectories thereof. It's my first time working with Rust, so I'm not really familiar with language best practices, but I tried to maintain the style of the codebase to the best of my abilities.

P.S. I forgot to turn my auto-formatter off while writing the patch! >.< Sorry for the massive diff, I can reformat before merge if desired.

Implemented feature request from #7. Implementation could use some cleanup for codebase aesthetics, but works properly on my machine. The logic permits storing executables in `layout_dir` and arbitrarily deep subdirectories thereof. It's my first time working with Rust, so I'm not really familiar with language best practices, but I tried to maintain the style of the codebase to the best of my abilities. P.S. I forgot to turn my auto-formatter off while writing the patch! >.< Sorry for the massive diff, I can reformat before merge if desired.

PerfectlyInternal commented 2026年03月06日 17:20:57 +01:00

Looks like the logic works. If you could undo the formatting changes that'd be great. Also, in the future, you can create a separate branch on your fork of the project instead of creating a whole fork for one PR. Thanks for the contribution!

Looks like the logic works. If you could undo the formatting changes that'd be great. Also, in the future, you can create a separate branch on your fork of the project instead of creating a whole fork for one PR. Thanks for the contribution!

allymander commented 2026年03月06日 18:46:15 +01:00

Reverted formatting, and tip noted!

Reverted formatting, and tip noted!