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.

Sign up for GitHub

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

Jump to bottom

タイトルからファイルを特定する処理の不備を修正 #34

Merged
merged 3 commits into from
Oct 31, 2023
Merged

タイトルからファイルを特定する処理の不備を修正 #34

merged 3 commits into from
Oct 31, 2023
+40 −22

Conversation

halkt
Copy link
Member

@halkt halkt commented Oct 31, 2023 •
edited
Loading

背景

  • create-draftやpull-draftアクションを実施する際にタイトルからファイルを取得する処理があるが, 数値のみのタイトル等の場合などでダブルクォートが含まれるとファイルをうまく取得できずエラーになるため修正を図る

実装内容

  • create-draft
    • entry_path をtitleから取得せずにblogsync postの返り値から取得する形に変更
  • pull-draft
    • 従来どおりtitleから取得する形に変更(将来的に変更する可能性はあるが現時点では従来と同じ形に)
  • 共通アクション
    • entry_pathは本アクション内で取得していたがそれぞれで取得方法が変わったため, 引数として受け取る形に変更＆action名を変更した

@halkt halkt requested a review from theoremoon October 31, 2023 02:55
theoremoon
theoremoon approved these changes Oct 31, 2023
View reviewed changes
Copy link
Member

@theoremoon theoremoon left a comment

Choose a reason for hiding this comment

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

良いと思います

@halkt halkt merged commit 2192b98 into main Oct 31, 2023
@halkt halkt deleted the fix-find-entry branch October 31, 2023 05:26
@hatenablog-workflows-app hatenablog-workflows-app bot mentioned this pull request Oct 31, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@theoremoon theoremoon theoremoon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@halkt @theoremoon