Skip to content

Add noTemplate config option#71

Open
evanisnor wants to merge 1 commit intomainfrom
eisnor/Add-noTemplate-config-option
Open

Add noTemplate config option#71
evanisnor wants to merge 1 commit intomainfrom
eisnor/Add-noTemplate-config-option

Conversation

@evanisnor
Copy link
Copy Markdown
Contributor

The --no-template flag added in #70 requires passing it on every sd new invocation. For users who always want raw commit bodies as PR descriptions, this adds a persistent config option.

Setting noTemplate: true in ~/.gh-stacked-diff/config.yaml (or via --config noTemplate=true) makes sd new skip template processing by default. The --no-template flag still works as an explicit override.

  • Add NoTemplate field to UserConfig and YamlConfig with file/flag merging in NewUserConfig()
  • Fall back to userConfig.NoTemplate in sd new when --no-template flag is not explicitly set
  • Document noTemplate in the --config help text and config.yaml example
  • Add 5 unit tests for config parsing and 1 integration test for config-driven sd new

Ticket: NONE

Feature flag(s): NONE

The `--no-template` flag added in #70 requires passing it on every `sd new` invocation. For users who always want raw commit bodies as PR descriptions, this adds a persistent config option.

Setting `noTemplate: true` in `~/.gh-stacked-diff/config.yaml` (or via `--config noTemplate=true`) makes `sd new` skip template processing by default. The `--no-template` flag still works as an explicit override.

* Add `NoTemplate` field to `UserConfig` and `YamlConfig` with file/flag merging in `NewUserConfig()`
* Fall back to `userConfig.NoTemplate` in `sd new` when `--no-template` flag is not explicitly set
* Document `noTemplate` in the `--config` help text and config.yaml example
* Add 5 unit tests for config parsing and 1 integration test for config-driven `sd new`

#### Ticket: NONE

#### Feature flag(s): `NONE`

<!--

| Before | After |
| ------------- | ------------- |
| <img src="" alt="" width="300"/> | <img src="" alt="" width="300"/> |
| <video src="" alt="" width="300"/> | <video src="" alt="" width="300"/> |

-->
@evanisnor evanisnor marked this pull request as ready for review April 16, 2026 13:34
@evanisnor evanisnor requested a review from slack-jallen April 16, 2026 13:34
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.

1 participant