search

joshrotenberg / adrs

Architectural Decision Record tool in Rust
http://joshrotenberg.com/adrs/
Apache License 2.0
19 stars 5 forks source link

bug: reading the .adr-dir file has inconsistencies #4

Closed joshrotenberg closed 6 months ago

joshrotenberg commented 6 months ago

I'm hoping to deprecate this in favor of something like an environment variable such as ADRS_DIRECTORY, but in the meantime, I see a bug where reading the string read from the file doesn't get the newline trimmed, and hence adrs doesn't think the directory exists. This definitely happens in the new command, so the short term fix is to trim whatever is read. This file gets read enough that the call should get abstracted out into adrs.rs at the very least.

joshrotenberg commented 6 months ago

Centralized and cleaned up the .adr-dir reading code, should be fixed now.