Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v2.4.1 #137

Merged
merged 10 commits into from
Nov 13, 2024
Merged

v2.4.1 #137

merged 10 commits into from
Nov 13, 2024

Conversation

DefinetlyNotAI
Copy link
Owner

Pull Request Template

Prerequisites

  • I have searched for duplicate or closed issues.
  • I have read the contributing guidelines.
  • I have followed the instructions in the wiki about contributions.
  • I have updated the documentation accordingly, if required.
  • I have tested my code with the --dev flag, if required.

PR Type

  • Bug fix
  • New feature
  • Refactoring
  • Documentation
    update
  • ⚠️ Breaking change ⚠️

Description

Ehh, fixed bugs that made half of the program not run

Motivation and Context

Many minor and major bugs

Credit

N/A

Issues Fixed

N/A

The __lib_class.py was failing to actually execute code due to a faulty shell code. Code affected are .bat, .exe and .ps1 that are purely executed from the Execute.script method, and Execute.__run_other_script method specifically - This does break how logging in the __lib_class.py works, so documents in wiki will be changed accordingly
Due to rehaul of __lib_class.py, threading has a failure bug `TypeError`, its fixed now
To increase performance
For some reason the _zipper.py didn't properly zip directories, and so half of the data that was grouped in folders, were deleted
For some reason the _zipper.py didn't properly zip directories, and so half of the data that was grouped in folders, were deleted
Fixed the issue of random operation completion message not adhering colorlog
Made it easier to maintain and in functions, fixed minor bugs, etc
@DefinetlyNotAI DefinetlyNotAI self-assigned this Nov 13, 2024
@DefinetlyNotAI DefinetlyNotAI added type/Documentation Improvements or additions to commentations request/Important New feature or request, top priority, for next update bug/High Something isn't working, it's broken! type:Code Related to the Code part type/Development Related to Development issues labels Nov 13, 2024
@pull-request-size pull-request-size bot added the size/XL Huge size pr label Nov 13, 2024
CODE/_zipper.py Show resolved Hide resolved
Copy link

codeclimate bot commented Nov 13, 2024

Code Climate has analyzed commit f714a5b and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

View more on Code Climate.

@DefinetlyNotAI DefinetlyNotAI merged commit bc1518a into main Nov 13, 2024
8 checks passed
@DefinetlyNotAI DefinetlyNotAI deleted the v2.4.1 branch November 13, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/High Something isn't working, it's broken! request/Important New feature or request, top priority, for next update size/XL Huge size pr type:Code Related to the Code part type/Development Related to Development issues type/Documentation Improvements or additions to commentations
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant