Fix/improve syspaths warning message [3006.x] #68444
Conversation
- Rename misleading variable 'typo_warning' to 'missing_vars_warning' - Update warning message to be more accurate about missing variables - Fix typo in warning message: 'dissolve' -> 'resolve' - Improve code readability and maintainability The variable name 'typo_warning' was misleading as it's actually about missing variables in _syspaths.py, not typos. This change makes the code more self-documenting and the warning message more helpful.
- Set sys.dont_write_bytecode = True in LazyLoaderReloadingTest.setUp() - Restore original setting in tearDown() to avoid side effects - Remove redundant remove_bytecode() calls in update_module() and rm_module() - Addresses TODO comments about .pyc files being slower on reload (~3x) This optimization prevents .pyc files from being written during test execution, which was causing performance issues when modules were reloaded quickly. The original TODO suggested this approach as being much faster than writing and then removing .pyc files.
|
Hi there! Welcome to the Salt Community! Thank you for making your first contribution. We have a lengthy process for issues and PRs. Someone from the Core Team will follow up as soon as possible. In the meantime, here's some information that may help as you continue your Salt journey. There are lots of ways to get involved in our community. Every month, there are around a dozen opportunities to meet with other contributors and the Salt Core team and collaborate in real time. The best way to keep track is by subscribing to the Salt Community Events Calendar. |
Backport of #68412 to 3006.x.
This PR cherry-picks the commits from the original PR and targets the oldest supported branch as requested.
Original PR: #68412