Refactor #49

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.

Sign up for GitHub

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

Jump to bottom

Open

Antosik wants to merge 6 commits into NodeBB-Community:master from ariastel:update

Antosik commented Sep 27, 2021 •

edited

Loading

Done

ESLint & related auto-fixes (4108de0, d385549)
Big refactor (977d142)
- Code was splatted into multiple parts for readability
- async/await
- A Deprecated request was replaced by the undici
- Misc directory structure updates
- Comments for code
Dependencies update (d0917d5)

P.S. For reviewers - please double-check everything 👀

Antosik added 6 commits

September 27, 2021 19:33


          chore: ESLint

4108de0


          fix: auto-fix ESLint errors

d385549


          feat: refactor

977d142


          feat: update deps & bump compatibility

d0917d5


          fix: acp - check for input existance

d4b7fa7


          fix: wrap acpScript

acb8a3b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet