这是indexloc提供的服务,不要输入任何密码
Skip to content

Fix eslint config pkg name #1302

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

Merged
merged 5 commits into from
Jun 1, 2022
Merged

Fix eslint config pkg name #1302

merged 5 commits into from
Jun 1, 2022

Conversation

helgee
Copy link
Contributor

@helgee helgee commented May 28, 2022

Otherwise the build starts failing if you change the scope from acme as instructed in the README.

Otherwise the build starts failing if you change the scope from `acme` as instructed in the README.
@vercel
Copy link

vercel bot commented May 28, 2022

@helgee is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@nathanhammond
Copy link
Contributor

Thanks for catching the incomplete rename!

kodiakhq bot pushed a commit that referenced this pull request Jun 1, 2022
…t. (#1317)

This makes it possible for a repository whose CI configuration is `--remote-only` to work when receiving pull-requests from forked repositories that do not have access to secrets configuration in CI.

Example failures:
- #1302: [Error](https://github.com/vercel/turborepo/runs/6681959655?check_suite_focus=true#step:8:56)
- vercel/vercel#7575: [Error](https://github.com/vercel/vercel/runs/6599439965?check_suite_focus=true#step:10:12)

One example workaround:
vercel/vercel#7873

However, since this does not prevent the build from running to completion, I do not believe that it should be a hard error. While in the process of addressing this I also discovered that, since the `httpCache` can remove itself, there is an alternative path to triggering a panic if the `httpCache` is the only cache you have enabled and it removes itself.

Using a `noopCache` as a cache of last resort (similar to the [upstream solution](thought-machine/please@e0b5e29)) resolves both of these issues.

Related to: #1169
@jaredpalmer jaredpalmer added the pr: automerge Kodiak will merge these automatically after checks pass label Jun 1, 2022
@kodiakhq kodiakhq bot merged commit eb7efc2 into vercel:main Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: automerge Kodiak will merge these automatically after checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants