Skip to content

Allow a prop format to be specified#61

Merged
aaronjorbin merged 24 commits intotrunkfrom
feature/allow-choice-of-format
Feb 6, 2024
Merged

Allow a prop format to be specified#61
aaronjorbin merged 24 commits intotrunkfrom
feature/allow-choice-of-format

Conversation

@desrosj
Copy link
Copy Markdown
Member

@desrosj desrosj commented Feb 6, 2024

What?

This updates the action to allow a repository to configure the specific formats they want included in the bot's comments.

Why?

Often times, only one format is relavent to a given repo. This also helps make the comment shorter. Including both formats makes it quite long.

How?

Using the formats input, all, git, or svn can now be specified. The default is git.

Fixes #38, #47.

@desrosj desrosj self-assigned this Feb 6, 2024
@desrosj desrosj force-pushed the feature/allow-choice-of-format branch from a2db431 to 131e335 Compare February 6, 2024 15:21
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 6, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core SVN

Core Committers: Use this line as a base for the props when committing in SVN:

Props desrosj, jorbin, noisysocks, gziolo.

GitHub Merge commits

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: desrosj <desrosj@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
Co-authored-by: noisysocks <noisysocks@git.wordpress.org>
Co-authored-by: gziolo <gziolo@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@desrosj desrosj requested a review from aaronjorbin February 6, 2024 18:22
@desrosj desrosj linked an issue Feb 6, 2024 that may be closed by this pull request
desrosj and others added 4 commits February 6, 2024 13:46
Co-authored-by: Aaron Jorbin <aaronjorbin@users.noreply.github.com>
Co-authored-by: Aaron Jorbin <aaronjorbin@users.noreply.github.com>
…/props-bot-action into feature/allow-choice-of-format
@desrosj desrosj requested a review from aaronjorbin February 6, 2024 19:03
@aaronjorbin aaronjorbin merged commit 64511b5 into trunk Feb 6, 2024
@aaronjorbin aaronjorbin deleted the feature/allow-choice-of-format branch February 6, 2024 21:05
pento pushed a commit to WordPress/wordpress-develop that referenced this pull request Feb 6, 2024
The new default is a git flavored version, while Core wants the SVN flavor. This update is required after WordPress/props-bot-action#61

Props desrosj, jorbin, noisysocks, gziolo.
See #60417.


git-svn-id: https://develop.svn.wordpress.org/trunk@57544 602fd350-edb4-49c9-b593-d223f7449a82
markjaquith pushed a commit to markjaquith/WordPress that referenced this pull request Feb 6, 2024
The new default is a git flavored version, while Core wants the SVN flavor. This update is required after WordPress/props-bot-action#61

Props desrosj, jorbin, noisysocks, gziolo.
See #60417.

Built from https://develop.svn.wordpress.org/trunk@57544


git-svn-id: http://core.svn.wordpress.org/trunk@57045 1a063a9b-81f0-0310-95a4-ce76da25c4cd
github-actions bot pushed a commit to platformsh/wordpress-performance that referenced this pull request Feb 6, 2024
The new default is a git flavored version, while Core wants the SVN flavor. This update is required after WordPress/props-bot-action#61

Props desrosj, jorbin, noisysocks, gziolo.
See #60417.

Built from https://develop.svn.wordpress.org/trunk@57544


git-svn-id: https://core.svn.wordpress.org/trunk@57045 1a063a9b-81f0-0310-95a4-ce76da25c4cd
aaronjorbin added a commit that referenced this pull request Feb 6, 2024
aaronjorbin added a commit that referenced this pull request Feb 6, 2024
This reverts commit 64511b5.

Where is the unprops bot when you need one?

Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
Co-authored-by: dmsnell <dmsnell@git.wordpress.org>
Co-authored-by: jeffpaul <jeffpaul@git.wordpress.org>
@aaronjorbin aaronjorbin restored the feature/allow-choice-of-format branch February 6, 2024 21:54
aaronjorbin added a commit that referenced this pull request Feb 8, 2024
Take 2.  

Fixes #38, #47.
See: #61, #63.

Co-authored-by: desrosj <desrosj@git.wordpress.org>
Co-authored-by: aaronjorbin <jorbin@git.wordpress.org>
Co-authored-by: noisysocks <noisysocks@git.wordpress.org>
Co-authored-by: gziolo <gziolo@git.wordpress.org>
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.

Only show the relevant props list Allow configuring the details included in the comment

2 participants