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

blogsyncの更新との協調について #27

Closed
Songmu opened this issue Oct 23, 2023 · 2 comments
Closed

blogsyncの更新との協調について #27

Songmu opened this issue Oct 23, 2023 · 2 comments

Comments

@Songmu
Copy link
Contributor

Songmu commented Oct 23, 2023

最近良くも悪くもblogsyncのメンテにやる気を出してしまっているのですが、逆に、こちらのワークフローとの互換問題が出やすくなっていると思います。

また、blogsync側の機能拡張してやったほうが良いのか、このワークフロー側でやったほうが良いのかの役割分担も検討したほうが良いでしょう。個人的所見としては、こちらのワークフローもちょっと頑張り過ぎで、blogsync側を拡張してやったほうが良いものを、ワークフロー側でblogysyncが想定していない使い方で無理に解決している分壊れやすくなっているとも思います。実際URL周りで最近エンバグしていました。 x-motemen/blogsync#99

このあたりはちょっとメンテナンス方針を協議したほうが良いように感じています。少なくとも、古いblogsyncバージョンに固定し続けるのはあまり良くないと思います。

また、最近入れた更新で、このワークフロー側で有用そうな変更をお知らせします。

blogsyncが標準出力に更新されたファイルパスを出すように

x-motemen/blogsync#107

entry_path=$(blogsync post ...)

上記のようにすれば、更新や新規作成されたエントリーのファイルパスが取れるようになったので、 create-draft.yaml でタイトル名からファイルを解決する挙動がありますが、それが不要になると思います。

下書きを除外してpullする

x-motemen/blogsync#110

blogsync pull --no-drafts

blogsync pull 後にDraftを消し去っているところがあると思いますが、 --no-drafts オプションで同様のことを実現できるようになりました。

@halkt
Copy link
Member

halkt commented Oct 25, 2023

ご連絡ありがとうございます!今後の機能拡張については個別に相談をさせていただこうと思います!

少なくとも、古いblogsyncバージョンに固定し続けるのはあまり良くないと思います。

こちらはその通りなので, 近日対応予定です!

@halkt
Copy link
Member

halkt commented Oct 27, 2023

@Songmu

blogsyncのバージョン固定の解除を行いました
#29

@halkt halkt closed this as completed Nov 1, 2023
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

No branches or pull requests

2 participants