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

Move @types/node to devDependencies #1475

Merged
merged 2 commits into from
Jul 13, 2022
Merged

Move @types/node to devDependencies #1475

merged 2 commits into from
Jul 13, 2022

Conversation

yejia-stripe
Copy link
Contributor

@yejia-stripe yejia-stripe commented Jul 13, 2022

Closes #1474

r? @dcr-stripe

Copy link
Contributor

@dcr-stripe dcr-stripe left a comment

Choose a reason for hiding this comment

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

LGTM!

Did we used to use that mocha reporter package for CI and now no longer? Seems like it from #730 but I can't see where it was in CI...

@stripe-ci stripe-ci assigned yejia-stripe and unassigned dcr-stripe Jul 13, 2022
@yejia-stripe
Copy link
Contributor Author

Oh hm I'm actually not sure either, it just didn't look like it was being used. I'll revert that part!

@yejia-stripe yejia-stripe merged commit 0b94dd6 into master Jul 13, 2022
yejia-stripe added a commit that referenced this pull request Jul 13, 2022
@richardm-stripe richardm-stripe deleted the yejia/clean-deps branch April 4, 2023 23:03
gurus00 pushed a commit to gurus00/stripe-node that referenced this pull request Feb 11, 2025
gurus00 pushed a commit to gurus00/stripe-node that referenced this pull request Feb 11, 2025
gurus00 pushed a commit to gurus00/stripe-node that referenced this pull request Feb 11, 2025
gurus00 pushed a commit to gurus00/stripe-node that referenced this pull request Feb 11, 2025
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.

Can @types/node be installed only on development mode?
2 participants