Closed naartjie closed 3 years ago
Thank you @naartjie
CC: @bwdmonkey @bobheadxi
I haven't updated the README yet. Please let me know if it's something you'd like to merge, and I'll add info the README.
I haven't updated the README yet. Please let me know if it's something you'd like to merge, and I'll add info the README.
Might be good to add in a separate PR, since I think this belongs to a separate category of configuration in general (something like "Global configuration"?) that could be grouped together (currently, config docs is sorted by the steps in the README)
Hey @naartjie , any word on updating this PR? I think this would be a great feature to have, but I'm hoping to do some more significant changes to this action soon that I'm worried might conflict with your work here, so I'm hoping to land this first!
Hi @bobheadxi. Thanks for the review, and sorry for letting it linger. I'll address your feedback and rebase.
Then as you say, we can update the README as part of a separate PR.
@bobheadxi I've made the changes. Let me know what you think.
This PR adds an optional
repository
input config, which allows to target other repositories.The value can be:
undefined
: use the current actionowner/repo
repo
: use current owner, and set repoowner/repo
: set both