Typos

Author: Septh

README.md

@@ -124,7 +124,7 @@ Set the `builtins` option to `false` if you'd like to use some shims/polyfills f
 #### builtinsPrefix?: 'add' | 'strip' | 'ignore' = 'add'
 How to handle the `node:` scheme used in recent versions of Node (i.e., `import path from 'node:path'`).
 - If `add` (the default, recommended), the `node:` scheme is always added if missing. In effect, this dedupes your imports of Node builtins by homogenizing their names to their schemed version.
-- If `strip`, the scheme is always removed. In effect, this dedupes your imports of Node builtins by homogenizing their names to their unschemed version. Schemed-only builtins like `node:test` are never stripped.
+- If `strip`, the scheme is always removed. In effect, this dedupes your imports of Node builtins by homogenizing their names to their scheme-less version. Schemed-only builtins like `node:test` are never stripped.
 - `ignore` will simply leave all builtins imports as written in your code.
 > _Note that scheme handling is always applied, regardless of the `builtins` options being enabled or not._
 
@@ -252,7 +252,7 @@ export default defineConfig({
 - This plugin now has a **peer-dependency** on Rollup `^3.0.0 || ^4.0.0`.<br />*If you need Rollup 2 support, please stick to v5.*
 
 #### Breaking changes in version 5
-- In previous versions, the `devDeps` option defaulted to `true`.<br>This was practical, but often wrong: devDependencies are meant just for that: being used when developping. Therefore, the `devDeps` option now defaults to `false`, meaning Rollup will include them in your bundle.
+- In previous versions, the `devDeps` option defaulted to `true`.<br>This was practical, but often wrong: devDependencies are meant just for that: being used when developing. Therefore, the `devDeps` option now defaults to `false`, meaning Rollup will include them in your bundle.
 - As anticipated since v4, the `builtinsPrefix` option now defaults to `'add'`.
 - The deprecated `prefixedBuiltins` option has been removed. Use `builtinsPrefix` instead.
 - `rollup-plugin-node-externals` no longer depends on the Find-Up package (while this is not a breaking change per se, it can be in some edge situations).

test/_common.ts

@@ -25,7 +25,7 @@ class MockPluginContext {
             buildStart = buildStart.handler
         if (typeof buildStart === 'function')
             return await buildStart.call(this as any, {} as NormalizedInputOptions)
-        throw new Error('Ooops')
+        throw new Error('Oops')
     }
 
     async resolveId(specifier: string, importer: string | undefined) {
@@ -34,7 +34,7 @@ class MockPluginContext {
             resolveId = resolveId.handler
         if (typeof resolveId === 'function')
             return await resolveId.call(this as any, specifier, importer, { attributes: {}, isEntry: typeof importer === 'string' ? false : true })
-        throw new Error('Ooops')
+        throw new Error('Oops')
     }
 
     error(err: string | RollupError): never {

test/builtins.test.ts

@@ -19,7 +19,7 @@ test("Does NOT mark Node builtins external when builtins=false", async t => {
     }
 })
 
-test("Does NOT mark Node builtins external when implicitely excluded", async t => {
+test("Does NOT mark Node builtins external when implicitly excluded", async t => {
     const context = await initPlugin({ exclude: [ 'path', 'node:fs' ]})
     for (const builtin of [ 'path', 'node:fs' ]) {
         t.like(await context.resolveId(builtin, 'index.js'), {
@@ -55,7 +55,7 @@ test("Removes 'node:' prefix when using builtinsPrefix='strip'", async t => {
     }
 })
 
-test("Does NOT remove 'node:test' prefix even with builtinsPrefix='add'", async t => {
+test("Does NOT remove 'node:test' prefix even with builtinsPrefix='strip'", async t => {
     const context = await initPlugin({ builtinsPrefix: 'strip' })
     for (const builtin of [ 'node:test' ]) {
         t.like(await context.resolveId(builtin, 'index.js'), {