“Fixed issue with ssl python libs,” or “Minor bugfixes.”
In other news, never work more than one person on a branch (that’s why we have them). Make a new related issue with its own branch and rebase whenever necessary, and don’t even think about touching main or dev with anything but a properly reviewed and approved PR (in case they aren’t already protected), or I’ll find and report you to the same authority that handles all the failed sudo requests!
Also, companies that disable rebasing are my bane. While you can absolutely do without, i much prefer to have less conflicts, cleaner branches and commits, easier method to pull in new changes from dev, overall better times for the reviewer, and the list goes on. Though, the intern rewriting multiple branches’ history which they have no business pushing to is also rather annoying.
I hate when helpful features like hook scripts aren’t enabled. Someone forgot to run a script before pushing their changes even though the readme clearly says to format, lint, and confirm unit tests all pass before pushing, and now the PR has 100 irrelevant whitespace changes that are going to be reverted by the next pull request. A pre-commit script could gate that and auto fix it but noooo, admins don’t want to turn on a feature they don’t use.
In other news, never work more than one person on a branch (that’s why we have them). Make a new related issue with its own branch and rebase whenever necessary, and don’t even think about touching main or dev with anything but a properly reviewed and approved PR (in case they aren’t already protected), or I’ll find and report you to the same authority that handles all the failed sudo requests!
Also, companies that disable rebasing are my bane. While you can absolutely do without, i much prefer to have less conflicts, cleaner branches and commits, easier method to pull in new changes from dev, overall better times for the reviewer, and the list goes on. Though, the intern rewriting multiple branches’ history which they have no business pushing to is also rather annoying.
I hate when helpful features like hook scripts aren’t enabled. Someone forgot to run a script before pushing their changes even though the readme clearly says to format, lint, and confirm unit tests all pass before pushing, and now the PR has 100 irrelevant whitespace changes that are going to be reverted by the next pull request. A pre-commit script could gate that and auto fix it but noooo, admins don’t want to turn on a feature they don’t use.