Skip to content

ci: ワークフロー統合と protobuf 検証ジョブの追加#281

Merged
FRICK-ELDY merged 4 commits into
mainfrom
ci/merge-workflows-proto-verify
Mar 27, 2026
Merged

ci: ワークフロー統合と protobuf 検証ジョブの追加#281
FRICK-ELDY merged 4 commits into
mainfrom
ci/merge-workflows-proto-verify

Conversation

@FRICK-ELDY

Copy link
Copy Markdown
Owner
  • ci.yml.ignore を ci.yml に統合(pull_request・concurrency・各ジョブ)
  • proto-verify: protoc / mix alchemy.gen.proto / format 後に git diff
  • Rust fmt、Logger.warning、生成 .pb.ex と周辺の mix format 修正

@FRICK-ELDY FRICK-ELDY self-assigned this Mar 27, 2026

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request primarily focuses on code formatting and style consistency across Elixir and Rust files. Key changes include updating deprecated Logger.warn/1 calls to Logger.warning/1 to align with modern Elixir standards, standardizing the syntax in generated Protobuf Elixir modules by adding parentheses to macro calls, and various whitespace and indentation adjustments to improve readability. I have no feedback to provide as the changes are stylistic or address deprecations.

@FRICK-ELDY FRICK-ELDY merged commit 1a59e3c into main Mar 27, 2026
12 checks passed
@FRICK-ELDY FRICK-ELDY deleted the ci/merge-workflows-proto-verify branch March 27, 2026 16:08
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.

1 participant