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

feature: add typings and build with typescript compiler #17

Merged
merged 3 commits into from
Jan 5, 2022

Conversation

sveisvei
Copy link
Contributor

@sveisvei sveisvei commented Jan 5, 2022

I needed typescript typings, so ended up using the typescript compiler to allow js and output sensible node compatible code.

@jancurn jancurn requested a review from gippy January 5, 2022 10:04
@mtrunkat mtrunkat requested review from jirimoravcik and mhamas and removed request for gippy January 5, 2022 10:41
@jirimoravcik
Copy link
Member

Hello @sveisvei, we actually have a TS config available, could you please make it so you extend it? The repository is here https://github.com/apify/apify-tsconfig

@sveisvei
Copy link
Contributor Author

sveisvei commented Jan 5, 2022

@jirimoravcik fixed

rimraf wanted glob as well.

Copy link

@mhamas mhamas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me, thanks @sveisvei

@jirimoravcik jirimoravcik merged commit 53a9512 into apify:master Jan 5, 2022
@jirimoravcik
Copy link
Member

@sveisvei New version is released as 1.0.1-beta.0

@sveisvei
Copy link
Contributor Author

sveisvei commented Jan 6, 2022

@jirimoravcik Thank you, will switch to this from my fork.

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.

3 participants