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

Demote some dependencies to devDependencies #52

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dzbarsky
Copy link

@dzbarsky dzbarsky commented Aug 30, 2024

It's currently not possible to install this package because it depends on a private repo...

➜  sdk git:(master) ✗ npm install @airtable/blocks -f                                                                                                                                                                                                                   ~/blocks/packages/sdk
npm WARN using --force Recommended protections disabled.
npm WARN ERESOLVE overriding peer dependency
npm WARN While resolving: @airtable/[email protected]
npm WARN Found: peer typescript@">=2.7" from [email protected]
npm WARN packages/cli-next/node_modules/ts-node
npm WARN   dev ts-node@"8" from @airtable/[email protected]
npm WARN   packages/cli-next
npm WARN 
npm WARN Could not resolve dependency:
npm WARN peer typescript@">=2.7" from [email protected]
npm WARN packages/cli-next/node_modules/ts-node
npm WARN   dev ts-node@"8" from @airtable/[email protected]
npm WARN   packages/cli-next
npm WARN ERESOLVE overriding peer dependency
npm WARN ERESOLVE overriding peer dependency
npm ERR! code 128
npm ERR! An unknown git error occurred
npm ERR! command git --no-replace-objects ls-remote ssh://[email protected]/hyperbase/eslint-plugin-airtable.git
npm ERR! ERROR: Repository not found.
npm ERR! fatal: Could not read from remote repository.
npm ERR! 
npm ERR! Please make sure you have the correct access rights
npm ERR! and the repository exists.

npm ERR! A complete log of this run can be found in: /Users/dzbarsky/.npm/_logs/2024-08-30T14_43_38_701Z-debug-0.log

While I was here:

  • the lodash dep is unused
  • removed around 500 files from the distribution that users probably don't care about. (By the way, the stories couldn't even work, because one of them depends on prettier which is a devDep...)

This needs the lockfile to be regenerated; I can't do that because of

➜  sdk git:(master) yarn install                                                                                                                                                                                                                                        ~/blocks/packages/sdk
yarn install v1.22.19
[1/4] 🔍  Resolving packages...
error Couldn't find package "@airtable-blocks-internal/changelog-publish@^1.0.2" required by "@airtable/[email protected]" on the "npm" registry.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
Error: Couldn't find package "@airtable-blocks-internal/changelog-publish@^1.0.2" required by "@airtable/[email protected]" on the "npm" registry.

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.

1 participant