search

dimfeld / svelte-maplibre

Svelte bindings for the MapLibre mapping library
https://svelte-maplibre.vercel.app
MIT License
283 stars 34 forks source link

Adding prepare file #63

Closed stuartlynn closed 9 months ago

stuartlynn commented 9 months ago

Adds a prepare script so that people can use a link to github for the project dependency as well as npm.

https://docs.npmjs.com/cli/v10/using-npm/scripts#prepare-and-prepublish

vercel[bot] commented 9 months ago

Someone is attempting to deploy a commit to a Personal Account owned by @dimfeld on Vercel.

@dimfeld first needs to authorize it.

dimfeld commented 9 months ago

I think the best change here might just be to change prepublishOnly to prepare instead. It would be nice if there was a version of this command that didn't run on a normal npm install in the package, but I haven't seen an option for that.

changeset-bot[bot] commented 9 months ago

⚠️ No Changeset found

Latest commit: 23c1f2b14e14b66570f7fda5abd07ab5c675f3d6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

stuartlynn commented 9 months ago

@dimfeld Changed prepublishOnly to prepare as you suggested. Think this should be good to merge noww