Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow a prop format to be specified #61

Merged
merged 24 commits into from
Feb 6, 2024

Conversation

desrosj
Copy link
Contributor

@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
src/github.js Outdated Show resolved Hide resolved
@desrosj desrosj force-pushed the feature/allow-choice-of-format branch from a2db431 to 131e335 Compare February 6, 2024 15:21
Copy link

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 linked an issue Feb 6, 2024 that may be closed by this pull request
src/github.js Outdated Show resolved Hide resolved
src/github.js Outdated Show resolved Hide resolved
src/github.js Outdated Show resolved Hide resolved
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
@aaronjorbin aaronjorbin merged commit 64511b5 into trunk Feb 6, 2024
6 checks passed
@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