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

4.1.0 has no d.ts files #811

Closed
pjc0247 opened this issue Jan 17, 2023 · 12 comments
Closed

4.1.0 has no d.ts files #811

pjc0247 opened this issue Jan 17, 2023 · 12 comments

Comments

@pjc0247
Copy link

pjc0247 commented Jan 17, 2023

https://www.npmjs.com/package/code-push?activeTab=explore

4.0.5 works fine.

@microsoft-github-policy-service
Copy link
Contributor

This issue has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs within 15 days of this comment.

1 similar comment
@microsoft-github-policy-service
Copy link
Contributor

This issue has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs within 15 days of this comment.

@pjc0247
Copy link
Author

pjc0247 commented May 16, 2023

a

@saurav0705
Copy link
Contributor

any update on this?

@saurav0705
Copy link
Contributor

Please find the pr for the above issue #827

@regalstreak
Copy link

bumping ^ @DmitriyKirakosyan

@DmitriyKirakosyan
Copy link
Contributor

@regalstreak , @saurav0705 , @pjc0247 ,

I tried to find the problem with d.ts files not being created using Node 12 and 16, but everything worked fine for me.

Can you tell me exactly what steps you took? This will help me understand the issue better.

Also, I looked at versions 4.0.5 and 4.1.0 and didn't see any big changes that could cause this problem. If you noticed something, please share.

@regalstreak
Copy link

@DmitriyKirakosyan try creating a new typescript project and adding the latest version of code-push, the package misses the declarations (checked in node_modules)

4.0.5 also didn't have the d.ts for me

@saurav0705
Copy link
Contributor

saurav0705 commented Oct 30, 2023

@DmitriyKirakosyan the issue the release is pushed with the npm run build command due to which no .d.ts files are generated and package is published with that command

I have raised a pr solving this issue for future can you please check #828

@velimir-jankovic
Copy link

@saurav0705 thank you for the help! We've fixed our internal release process, so next release will include definition files properly.

@victor-chan-groundswell
Copy link

victor-chan-groundswell commented Feb 23, 2024

I'm actually running into this right now, going from 4.0.5 to 4.2.0..... I double checked what's in the node_modules directory and the .d.ts files are definitely missing in the new version. I am using node 21.

With 4.05 after running yarn install...

image

With 4.20 (on a different branch) after running yarn install....

image

Seeing that the code is merged in Nov but the latest release is in August 2023... when should there be an ETA on this? We'd like to upgrade so that it gets rid of the vm2 dependency.

@DmitriyKirakosyan
Copy link
Contributor

hi @victor-chan-groundswell , We are planning to release a new version today. There will be another vulnerability fix included.

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

6 participants