Rename to kebab-case for root action inputs and outputs#668
Conversation
🦋 Changeset detectedLatest commit: 7d28c47 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
| has-changesets: | ||
| description: A boolean about whether there were changesets. Useful if you want to create your own publishing functionality. | ||
| pullRequestNumber: | ||
| pull-request-number: |
There was a problem hiding this comment.
that's pr-number in the /version. perhaps we should match it here?
https://github.com/changesets/action/blob/a35a66bd1ec69b007f5cef9646f34a13eb610a41/src/version/index.ts#L56C19-L56C28
note that it's also missing in the declared outputs of that subaction:
Line 40 in a35a66b
There was a problem hiding this comment.
Yeah we should align the names eventually. In this PR I'd like to focus on kebab-casing only
| required: false | ||
| default: true | ||
| commitMode: | ||
| commit-mode: |
There was a problem hiding this comment.
@beeequeue noticed (rightfully so) this isn't exactly tightly-coupled to committing. But I'm not sure if we have the appetitete to rename this stuff right now
| publish: | ||
| description: "The command to use to build and publish packages" | ||
| required: false | ||
| version: |
There was a problem hiding this comment.
A little bit out of this PR's scope but I feel those should be renamed to publish-script and version-script
There was a problem hiding this comment.
Yeah I can follow up with this later
Could also rename some to match the new version/publish sub-action names, but I decided to make this change step by step.