Skip to content

chore: update line-bot-sdk-go to v8.23.0 and refactor handler - #33

Merged
kkdai merged 1 commit into
masterfrom
chore/update-deps-and-refactor
Oct 4, 2026
Merged

kkdai merged 1 commit into
masterfrom
chore/update-deps-and-refactor

Conversation

@kkdai

@kkdai kkdai commented Oct 4, 2026

Copy link
Copy Markdown
Owner
  • Bump SDK to v8.23.0 (requires Go 1.26) and update CI actions
  • Split main() into callbackHandler, handleEvent and replyText
  • Drop duplicated reply code; update README

- Bump SDK to v8.23.0 (requires Go 1.26) and update CI actions
- Split main() into callbackHandler, handleEvent and replyText
- Drop duplicated reply code; update README
@kkdai
kkdai requested a balanced review from Copilot October 4, 2026 05:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The dependency, CI, documentation, and behavior-preserving handler changes are consistent and complete.

Review effort: Balanced
Findings: None

What changed in this PR

Updates the LINE Bot SDK and Go toolchain while simplifying webhook event handling.

Changes:

  • Upgrade to SDK v8.23.0 and Go 1.26.
  • Extract callback, event-handling, and text-reply helpers.
  • Update CI actions and documentation.
File Description
README.md Documents new requirements and handler structure.
main.go Refactors webhook processing and reply logic.
go.mod Updates Go and SDK versions.
go.sum Updates SDK checksums.
.github/​workflows/​go.yml Updates actions and derives Go version from go.mod.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kkdai
kkdai merged commit 16fb58e into master Oct 4, 2026
2 checks passed
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.

2 participants