You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I wanted to open a discussion on the possibility of making the Shepherd "branding" during PR creation optional via env var / CLI option. The commit title in particular breaks things looking for semantic commit messages.
The text was updated successfully, but these errors were encountered:
@jugaltheshah, I'm definitely on board with this. I'd be happy to review a PR for this, if you'd like to take this on.
Another thing to consider and potential for future work. I think we'll need to implement configuration management that goes beyond env vars. That is, allow users to configure the behavior of Shepherd via some config file, say .shepherdrc.json with prioritization given to values set in .shepherdrc.json over env vars.
ROI:
Persistence of configurations across terminal sessions
I’m on board with that too. It might be good to just remove the prefix in Shepherd and rely on shepherd.yml specifying the title in full.
Agree with a better config system too. I wrote some thoughts on this here: #91
My vision is a unified “cascading” config system, where every option can either be set at a global level (e.g., ~/shepherd.yml), a migration file, or as command line options (with precedence working in that order too). I think a bifurcated system where there’s a separate set of “config” from migration options you set in shepherd.yml might make sense too, but I think it will be hard to predict what people will want to control globally vs on a per-migration basis.
I wanted to open a discussion on the possibility of making the Shepherd "branding" during PR creation optional via env var / CLI option. The commit title in particular breaks things looking for semantic commit messages.
The text was updated successfully, but these errors were encountered: