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

Marketplaceに公開するためのアクションの作成 #63

Merged
merged 1 commit into from
Apr 14, 2023
Merged

Marketplaceに公開するためのアクションの作成 #63

merged 1 commit into from
Apr 14, 2023

Conversation

theoremoon
Copy link
Contributor

#62 の通り、blogsyncの公式アクションがGitHub Marketplaceにあって、ユーザは自分のworkflowに uses: x-motemen/blogsync@v1 のように書くだけ、となっていると便利と思って、https://github.com/kayac/ecspresso/blob/v1/action.yml を参考に action.yml を作っています。

Copy link
Member

@Songmu Songmu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

カスタムアクションの要望と作成までありがとうございます。
ただ、残念ながらこれでは対応しきれていないケースがあります。

  • uses: x-motemen/blogsync@v1@v1 の部分はGITREFです。なので、 v1というtagかbranchが存在している必要がありますが、blogsyncには存在しません。escpressoの場合は、 v1, v2といったブランチ名を採用しているので、 kayac/ecspresso@v2 とすれば最新版を使えるのですが、blogsyncの場合はmasterブランチがメインブランチであるため uses: x-motemen/blogsync@master とする必要があるでしょう。
  • x-motemen/blogsync@v1 のようにしたい場合、個人的にはブランチ名よりタグが良いと思っているので、blogsyncのリリース時にv1タグを打ち直す(force pushする)方法が良いでしょう。
  • 細かい話で対応は必須ではありませんが、このactions.ymlはOSとARCHが決め打ちになっています。
  • 実は uses: x-motemen/blogsync@master とするだけであれば、Marketplaceに公開しなくても実行可能です。Marketplaceで検索されないだけです。
    • また、Marketplaceに公開したい場合は、action.ymlにname, description, brandingセクションが追加で必要です

このあたりもう少し整えていただいても構いません。また、僕の方で、このブランチをベースにして手直しをさせてもらうかもしれません。(このcontribution自体は残したいので)

@Songmu Songmu merged commit 0cc52e1 into x-motemen:master Apr 14, 2023
@github-actions github-actions bot mentioned this pull request Apr 14, 2023
@Songmu
Copy link
Member

Songmu commented Apr 14, 2023

v0.13.2 としてリリースしました。試してみてください

@theoremoon
Copy link
Contributor Author

#64 とリリースありがとうございました。早速便利に使わせてもらっています!

@theoremoon theoremoon deleted the create-action-yml branch April 18, 2023 02:06
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.

None yet

2 participants