Bunch of basic usabilitychanges #2
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
eicchen02
commented
Oct 5, 2024
- Added .gitignore
- removed send2trash with os.remove to avoid filling recycle bin (also changed readme to reflect this)
- Added note that ffmeg, avapi only works on linux (lmk if wrong)
- Check if temp_aud exists, auto makes it if not now
- Remakes temp_aud folder for every video to prevent clutter from interrupted runs
- Temp audio files have been renamed to use video name
- Added logging function to certain parts of function
- Shows that program is still running
- Might replace with a bar in the future
- Logs to /logs and console, console logging can be disabled
… recycling bin, updated documentation to reflect that Added todo comments in code for own use
…d runs Temp files now use video name as titles
|
LGTM 🫡 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.