-
Notifications
You must be signed in to change notification settings - Fork 26
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
rename repo to dagger-for-github #22
Conversation
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Done! Thank you! |
@aluzzardi I merge this one and will debrief with @grouville about the process. |
@grouville We might also change the action name: Line 1 in 2e5f57a
to Atm still https://github.com/marketplace/actions/dagger-action |
In the context of Github marketplace, it kind of makes sense that it would be https://github.com/marketplace/actions/dagger seems more natural
Does that compatible with keeping the dagger-for-github name elsewhere, or will it confuse users?
… On Feb 25, 2022, at 12:27, CrazyMax ***@***.***> wrote:
@grouville We might also change the action name:
https://github.com/dagger/dagger-for-github/blob/2e5f57a90b1fd846ab90c865cd143aef6452000a/action.yml#L1
to Dagger for GitHub so link in the marketplace would be https://github.com/marketplace/actions/dagger-for-github.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.
|
We can't unfortunately. Name in marketplace is tied to username/orgname on GitHub so we can't pick |
Makes sense. I think most people (me included) will copy-paste the workflow yaml anyway. Thanks! |
do you prefer appearing as |
For consistency, I would prefer |
fixes #21
@shykes @aluzzardi would temporary need admin rights to the repo to rename it when merged and also check everything is ok in the actions marketplace.
Signed-off-by: CrazyMax crazy-max@users.noreply.github.com