JAForbes / pr-release

Manage releases with pull request descriptions and metadata instead of config.
6 stars 1 forks source link

Strip headers and todos from change description #289

Closed JAForbes closed 2 years ago

JAForbes commented 2 years ago

While testing out mithril I ended up with a lot of Description: the description and Context and Motivation: the context and motivation. It made me think, when would a header really be suitable for change copy given it is two-ish sentences and we already extract the PR title?

So I'll push a fix to remove headers.

I also saw checklists too, and yeah I think turning todos into change descriptions doesn't make sense either.