Skip to content
New issue

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

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

Already on GitHub? # to your account

refactor(anime-dl): better error handling/processing #25

Merged
merged 1 commit into from
May 28, 2022

Conversation

vinnyA3
Copy link
Owner

@vinnyA3 vinnyA3 commented May 28, 2022

Description

  • handle errors better (delegate thrown errors to initial caller)

    • catch block in bin entry
  • move consts to constants file

  • create anime-dl/locales file for strings & generated strings - avoid logic
    pollution & keeps things constant

Related Issues

Motivation and Context

Cleaner errors, better reasoning. Moves constants to files for easier & more
transparent updating.

How To Test It?

  • Environment: Linux, MacOS
  • Node Version: v16.13.1
  • YT-DLP Version: 2021.12.27

Steps:

Visual reference

No visual changes made

Checklist

  • I have tested on a supported environment: Linux
  • I have tested using the project's Node version: v16.13.1
  • [n/a] I have added tests to cover my changes.
  • [n/a] I have updated the documentation accordingly.

* handle errors better (delegate thrown errors to initial caller)
* move constants to files
* create locales file to strings & generated strings - avoid logic
  pollution & keeps things constant

[#18]
@vinnyA3 vinnyA3 merged commit f402268 into master May 28, 2022
@vinnyA3 vinnyA3 deleted the feat/improve-error-handling branch May 28, 2022 20:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant