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

Modernize the app #300

Open
Bam92 opened this issue Oct 28, 2020 · 1 comment
Open

Modernize the app #300

Bam92 opened this issue Oct 28, 2020 · 1 comment

Comments

@Bam92
Copy link

Bam92 commented Oct 28, 2020

I came across this project at random when looking for a tool to create a database schema.

I tried to look at the code and I've noticed two issues:

  • there's no tests (unit or whatever)
  • old js
    So, I wish we can make the app a bit more modern by adding tests and adopting modern JavaScript.
@ondras
Copy link
Owner

ondras commented Nov 24, 2020

* So, I wish we can make the app a bit more modern by adding tests and adopting modern JavaScript.

Feel free! I am too busy these days, so re-factoring the app is out of my scope. But I would probably accept a PR that adjusts the code to modern JavaScript, whatever that currently means. Just do not introduce any other dependencies.

(Unfortunately, I do not think this would actually present any kind of improvements for users, but whatever.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants