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

Migrate to typescript #156

Closed
wants to merge 4 commits into from
Closed

Migrate to typescript #156

wants to merge 4 commits into from

Conversation

anbraten
Copy link
Contributor

@anbraten anbraten commented Mar 8, 2024

I had a look if it would make sense to migrate this library to typescript.
Would be interested if this is a thing you would consider.
If so I would be happy to finish this migration.

@anbraten anbraten changed the title Ts Migrate to typescript Mar 8, 2024
@brendannee
Copy link
Member

This would be great! I'd love to have it migrated to typescript.

If you want to finish this migration and let me know, I can review and help in any way.

@brendannee
Copy link
Member

Let me know if you need any help or have questions as part of this migration - I'm very interested to migrate to Typescript and appreciate your efforts so far.

@anbraten
Copy link
Contributor Author

I did some improvements based on this repo in my own project for now: https://github.com/kiel-live/transport-hub/tree/main/importer

Next I would try to bring those improvements back here. Somehow I also finally solved the out-of-memory issue I had before, but I still have no idea what was the issue before.

@brendannee
Copy link
Member

I just pushed an updated version https://github.com/BlinkTagInc/node-gtfs/releases/tag/4.13.0 which converts this library to Typescript and updates several other things. Check it out and let me know if you have suggestions on how to improve it or see any issues.

@anbraten anbraten closed this Jul 11, 2024
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

Successfully merging this pull request may close these issues.

2 participants