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
{{ message }}
Repository navigation
Create changelog entry generators for common use cases #159
Important question: what naming convention should we use? @changesets/github-changelogs? @changesets/changelog-github? Do we care about having a naming convention for non-changesets scoped changelog entry generators?
I vote @changesets/changelog-github for ones we publish but also I don't think trying to enforce anything strong here is important.
The BB one can probably just be a standardising of the Atlaskit one.
Do we want to do anything with the format of these files before doing this? For example making a single getChangelogEntry function rather than the currently awkward line-by-line option?
Already written, but the 'basic' ones that come with the CLI is the other option (no git, no extras)
I'd like this to also not be the default, so apply-release-plan can stop supplying the hashes, but do worry about over-complicating these functions if we make function authors get that info.
I vote @changesets/changelog-github for ones we publish but also I don't think trying to enforce anything strong here is important.
The BB one can probably just be a standardising of the Atlaskit one.
Both SGTM
Do we want to do anything with the format of these files before doing this? For example making a single getChangelogEntry function rather than the currently awkward line-by-line option?
I'm think this is probably a later thing, mainly because I don't think we really have a big need for it
github
git (uses the git hashes, but doesn't fetch out)
I'd like this to also not be the default, so apply-release-plan can stop supplying the hashes, but do worry about over-complicating these functions if we make function authors get that info.
SGTM
My general thoughts on most of these things is: Let's get the ideal behavior for most consumers and once we have the behavior we want, let's find the right internals/parts of the API that people use less.
Fair. I know KS asked about more control here at one point and were disappointed they couldn't control the full thing.
Let's get the ideal behavior for most consumers and once we have the behavior we want, let's find the right internals/parts of the API that people use less
Activity
emmatown commentedon Sep 17, 2019
Important question: what naming convention should we use?
@changesets/github-changelogs?@changesets/changelog-github? Do we care about having a naming convention for non-changesets scoped changelog entry generators?Noviny commentedon Sep 18, 2019
Blat of random thoughts:
I vote
@changesets/changelog-githubfor ones we publish but also I don't think trying to enforce anything strong here is important.The BB one can probably just be a standardising of the Atlaskit one.
Do we want to do anything with the format of these files before doing this? For example making a single
getChangelogEntryfunction rather than the currently awkward line-by-line option?Already written, but the 'basic' ones that come with the CLI is the other option (no git, no extras)
I'd probably split `github into:
Noviny commentedon Sep 18, 2019
I'd like this to also not be the default, so
apply-release-plancan stop supplying the hashes, but do worry about over-complicating these functions if we make function authors get that info.emmatown commentedon Sep 18, 2019
Both SGTM
I'm think this is probably a later thing, mainly because I don't think we really have a big need for it
SGTM
My general thoughts on most of these things is: Let's get the ideal behavior for most consumers and once we have the behavior we want, let's find the right internals/parts of the API that people use less.
Noviny commentedon Sep 18, 2019
Fair. I know KS asked about more control here at one point and were disappointed they couldn't control the full thing.
SGTM - action plan is go.
Noviny commentedon Oct 25, 2019
This is done sans BB one right? That is probably a task for ATL people to resolve
emmatown commentedon Oct 25, 2019
Yes