fix(vercel,netlify): Prevent NFT from scanning users directory

[wobsoriano]

Changes

This PR adds /users/** to ignored directories being scanned by @vercel/nft. This happens when:

1. You have a /users string somewhere in your codebase and your app is located in /Users/** directory
2. You're using pnpm and the package/module that has that /users string is not added to node_modules because of pnpm symlinks
3. @vercel/nft thinks the /users string is a directory and scans it
4. Throws an error because of permissions

Here's a complete convo from discord - https://discord.com/channels/830184174198718474/1281031671495397437

Testing

I tested it locally via pnpm patch, and run pnpm build. It works:

Docs

[changeset-bot]

🦋 Changeset detected

Latest commit: 303d78c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 24 packages

Name	Type
@astrojs/netlify	Patch
@astrojs/vercel	Patch
@test/netlify-hosted-astro-project	Patch
@test/astro-vercel-basic	Patch
@test/astro-vercel-function-per-route	Patch
@test/astro-vercel-image	Patch
@test/vercel-isr	Patch
@test/vercel-max-duration	Patch
@test/vercel-edge-middleware-with-edge-file	Patch
@test/vercel-edge-middleware-without-edge-file	Patch
@test/astro-vercel-no-output	Patch
@test/astro-vercel-prerendered-error-pages	Patch
@test/astro-vercel-redirects-serverless	Patch
@test/astro-vercel-redirects	Patch
@test/vercel-server-islands	Patch
@test/astro-vercel-serverless-prerender	Patch
@test/astro-vercel-serverless-with-dynamic-routes	Patch
@test/astro-vercel-static-assets	Patch
@test/astro-vercel-static	Patch
@test/vercel-streaming	Patch
@test/astro-vercel-with-speed-insights-enabled-output-as-server	Patch
@test/astro-vercel-with-speed-insights-enabled-output-as-static	Patch
@test/astro-vercel-with-web-analytics-enabled-output-as-static	Patch
vercel-hosted-astro-project	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[alexanderniebuhr]

I think this shouldn't have any side-effects

[matthewp]

Have another PR #381 which I think fixes this in a better way, but making the base be the workspace root.

[wobsoriano]

Have another PR #381 which I think fixes this in a better way, but making the base be the workspace root.

Let’s do it!

[wobsoriano]

Closing this in favor of #381

[alexanderniebuhr]

Still thank you for the contribution. We appreciate it a lot. Hope to see you around the repository in the future :)