Skip to content

Conversation

@DayTF
Copy link
Contributor

@DayTF DayTF commented Feb 6, 2026

Definition of Done

General

  • Write an explicit title for the Pull Request, following Conventional Commits specification
  • Test manually the implemented changes
  • Validate the code quality (indentation, syntax, style, simplicity, readability)

Security

  • Consider the security impact of the changes made

@qltysh
Copy link

qltysh bot commented Feb 6, 2026

1 new issue

Tool Category Rule Count
qlty Structure Function with high complexity (count = 6): send_schema_to_server 1

'Error',
'Failed to send schema: cannot reach ForestAdmin server'
)
end
Copy link

Choose a reason for hiding this comment

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

Function with high complexity (count = 6): send_schema_to_server [qlty:function-complexity]

Copy link
Member

@matthv matthv left a comment

Choose a reason for hiding this comment

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

LGTM

@DayTF DayTF merged commit d309621 into main Feb 9, 2026
35 checks passed
@DayTF DayTF deleted the fix/log-error-on-post-apimap-failure branch February 9, 2026 09:46
forest-bot added a commit that referenced this pull request Feb 9, 2026
## [1.23.3](v1.23.2...v1.23.3) (2026-02-09)

### Bug Fixes

* **apimap:** log error on post apimap failure ([#262](#262)) ([d309621](d309621))
@forest-bot
Copy link
Member

🎉 This PR is included in version 1.23.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants