Skip to content

Fix linter errors#61

Merged
BudzikM merged 109 commits intomasterfrom
fix/linter_errors
Feb 24, 2026
Merged

Fix linter errors#61
BudzikM merged 109 commits intomasterfrom
fix/linter_errors

Conversation

@BudzikM
Copy link
Contributor

@BudzikM BudzikM commented Feb 16, 2026

  • fix linter errors
  • add error for function to avoid silent errors

@BudzikM BudzikM self-assigned this Feb 16, 2026
@BudzikM BudzikM changed the title [MERGE update/modules first] Fix linter errors Fix linter errors Feb 16, 2026
Copy link

@indivisus indivisus bot left a comment

Choose a reason for hiding this comment

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

Review completed in 10m.

Copy link

@indivisus indivisus bot left a comment

Choose a reason for hiding this comment

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

Review completed in 10m.

### 1 unresolved finding from previous reviews
Sev File Finding
P1 message.go Unchecked int64->int downcasts can overflow on arm builds

@BudzikM
Copy link
Contributor Author

BudzikM commented Feb 17, 2026

@indivisus approve this PR

Copy link

@indivisus indivisus bot left a comment

Choose a reason for hiding this comment

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

Review completed in 8m.

Copy link

@indivisus indivisus bot left a comment

Choose a reason for hiding this comment

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

Review completed in 9m.

Copy link

@indivisus indivisus bot left a comment

Choose a reason for hiding this comment

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

Review completed in 8m.

@sergiuszm sergiuszm self-requested a review February 24, 2026 09:31
@BudzikM BudzikM merged commit 3ebfb80 into master Feb 24, 2026
4 checks passed
@BudzikM BudzikM deleted the fix/linter_errors branch February 26, 2026 16:26
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.

3 participants