-
Notifications
You must be signed in to change notification settings - Fork 10
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
fix bugs in TypeScript code, add missing dependency and updater workflow #99
fix bugs in TypeScript code, add missing dependency and updater workflow #99
Conversation
Oh, Thanks for your help. I will take a look and a test this week. Because I am quite busy recently. Actually I merged some helper scripts from my other repository to improve this extension. And I am gonna to optimize the parser related code in the next version. |
Thanks 👍🏼 I highly appreciate your quick response and positive feedback |
7ebb676
to
a210fdb
Compare
…ested in code review
@AdrianDsg Sorry for waiting so long time, I was quite busy for working and personal things recently. Thanks for your help so much. |
Hi @hangxingliu,
I noticed you been working on a refactoring of the TypeScript code (of the downloaders) recently.
I really like the new structure it's so much easier to understand for me.
I tested the new code and noticed that the (dev) dependency
turndown-plugin-gfm
was apparently missing. So I added that one.I also removed some of the code that I unfortunately forgot to remove after my last debugging session in #90.
I then created a GitHub Actions workflow to ease the updating of the JSON assets.
It would run on the first day of the month and open a pull request with the changes.
Please see AdrianDsg#2 for an example. It's just an idea of mine to support you and the project.