Skip to content
This repository was archived by the owner on Feb 7, 2026. It is now read-only.

chore(deps): update dependency cpy-cli to v4#647

Merged
losalex merged 5 commits intomainfrom
fixCpyCli
Sep 13, 2022
Merged

chore(deps): update dependency cpy-cli to v4#647
losalex merged 5 commits intomainfrom
fixCpyCli

Conversation

@sofisl
Copy link
Copy Markdown
Contributor

@sofisl sofisl commented Aug 10, 2022

No description provided.

@sofisl sofisl requested review from a team August 10, 2022 22:30
@product-auto-label product-auto-label Bot added the size: xs Pull request size is extra small. label Aug 10, 2022
@product-auto-label product-auto-label Bot added the api: logging Issues related to the googleapis/nodejs-logging-bunyan API. label Aug 10, 2022
@losalex losalex changed the title try fixing cpy-cli fix: try fixing cpy-cli Aug 11, 2022
Copy link
Copy Markdown
Contributor

@losalex losalex left a comment

Choose a reason for hiding this comment

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

System test still fails with "error TS2307: Cannot find module './types/core' or its corresponding type declarations.". I wonder if we really need to update cp-cli - it seems that this change requires extra dependency, correct?

Copy link
Copy Markdown
Contributor

@losalex losalex left a comment

Choose a reason for hiding this comment

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

This still fails...

@sofisl
Copy link
Copy Markdown
Contributor Author

sofisl commented Aug 18, 2022

@losalex, this is an attempt at updating this dependency: #598

I'm not sure how to fix it, it seems like this ugprade is buggy: sindresorhus/cpy-cli#35 (there are more as well). I haven't had the chance to come back and look at this yet, if you do have cycles to upgrade it correctly I'd greatly appreciate it!

@product-auto-label product-auto-label Bot added the stale: old Pull request is old and needs attention. label Sep 10, 2022
@sofisl sofisl changed the title fix: try fixing cpy-cli chore(deps): update dependency cpy-cli to v4 Sep 13, 2022
Copy link
Copy Markdown
Contributor

@losalex losalex left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this - much appreciated!

@losalex losalex merged commit 266b392 into main Sep 13, 2022
@losalex losalex deleted the fixCpyCli branch September 13, 2022 22:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: logging Issues related to the googleapis/nodejs-logging-bunyan API. size: xs Pull request size is extra small. stale: old Pull request is old and needs attention.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants