Skip to content

fix: added fix for ConfigManager & fixed tests - #114

Merged
NikSimDev merged 2 commits into
masterfrom
bug/110-config-tests
Aug 14, 2026
Merged

fix: added fix for ConfigManager & fixed tests#114
NikSimDev merged 2 commits into
masterfrom
bug/110-config-tests

Conversation

@lev2p1

@lev2p1 lev2p1 commented Aug 14, 2026

Copy link
Copy Markdown
Owner

Pull Request

Related Issues:

Type of Change:

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Performance
  • Refactoring
  • Tests

Description:
Fixed issues related to locking an already locked mutex. Added minor fixes for the Configuration Manager: autosave is now disabled by default.

Breaking Changes:
(If any, describe what breaks and how to migrate)

Checklist:

  • Code compiles
  • Tests pass
  • Documentation updated
  • Changelog updated
  • No sensitive data exposed

@lev2p1
lev2p1 requested a review from NikSimDev August 14, 2026 22:03
@lev2p1 lev2p1 linked an issue Aug 14, 2026 that may be closed by this pull request
3 tasks

@NikSimDev NikSimDev left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good work

@NikSimDev
NikSimDev merged commit aaa2ddb into master Aug 14, 2026
@lev2p1
lev2p1 deleted the bug/110-config-tests branch August 15, 2026 19:13
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.

[BUG]: ConfigManager tests failed

2 participants