Skip to content

chore: adjust log level#1297

Merged
GabrielePicco merged 1 commit into
masterfrom
chore/adjust-log-level
Jun 4, 2026
Merged

chore: adjust log level#1297
GabrielePicco merged 1 commit into
masterfrom
chore/adjust-log-level

Conversation

@GabrielePicco
Copy link
Copy Markdown
Collaborator

@GabrielePicco GabrielePicco commented Jun 4, 2026

Summary by CodeRabbit

  • Chores
    • Updated internal logging level for transaction send failures to improve log clarity without affecting system functionality or public APIs.

@GabrielePicco GabrielePicco merged commit 568924d into master Jun 4, 2026
6 of 7 checks passed
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jun 4, 2026

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: f2471629-8b80-4c72-9f20-1ab26491f03f

📥 Commits

Reviewing files that changed from the base of the PR and between dd4b93f and 17908ad.

📒 Files selected for processing (1)
  • magicblock-services/src/actions_callback_service.rs

📝 Walkthrough

Walkthrough

This PR modifies logging behavior in the actions callback service. The import statement is updated to use the info tracing level instead of error, and the corresponding log statement in the callback transaction send loop is changed from error! to info! when a send fails. No API signatures or transaction-building logic is altered.

Suggested reviewers

  • bmuddha
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/adjust-log-level

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@GabrielePicco GabrielePicco deleted the chore/adjust-log-level branch June 4, 2026 14:14
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