Skip to content

Slight changes to template handling#48

Merged
joshrotenberg merged 1 commit intomainfrom
template-updates
Feb 9, 2025
Merged

Slight changes to template handling#48
joshrotenberg merged 1 commit intomainfrom
template-updates

Conversation

@joshrotenberg
Copy link
Owner

Some minor changes to the new template feature:

  1. Changed the environment variable option to be ADRS_TEMPLATE instead of ADRS_TEMPLATE_DIR since the arg points to the file itself
  2. Changed the flag to be lowercase '-t'
  3. Changed the behavior to be the following:
    • if a file is explicitly specified via the flag or ADRS_TEMPLATE, that file is used as the template
    • if no flag or env is specified, use $adr_dir/templates/template.md
    • if no #adr_dir/templates/template.md exists, use the built in default template

Fixes #47.

@lbenezriravin Let me know what you think about these changes when you have time.

these are kinda lame but oh well
@joshrotenberg
Copy link
Owner Author

Going for it ...

@joshrotenberg joshrotenberg merged commit 07def28 into main Feb 9, 2025
16 checks passed
@joshrotenberg joshrotenberg deleted the template-updates branch February 9, 2025 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

If ADRS_TEMPLATE_DIR is set or found in doc/adr, report the location in the config command

1 participant