Skip to content

chore: remove deprecated @types/glob dependency - #477

Merged
domoritz merged 1 commit into
apache:mainfrom
kentkwu:chore/remove-types-glob
Sep 11, 2026
Merged

chore: remove deprecated @types/glob dependency#477
domoritz merged 1 commit into
apache:mainfrom
kentkwu:chore/remove-types-glob

Conversation

@kentkwu

@kentkwu kentkwu commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Summary

@types/glob is a deprecated stub package that has had no effect on this repo for some time. glob v9 and later ship their own type definitions, and npm marks every published version of the stub as deprecated with the message "glob provides its own type definitions, so you do not need this installed."

  • Remove @types/glob from devDependencies, along with its transitive @types/minimatch
  • No source changes: the two glob consumers, bin/integration.ts and gulp/test-task.js, both use the v9+ named-export API that only the bundled types describe

Confirmed with tsc --traceResolution that import { glob } from 'glob' resolves to node_modules/glob/dist/esm/index.d.ts from glob 13.0.0, both before and after the change, so the stub was already being ignored.

Test Plan

  • tsc --noEmit --strict against a glob import, clean
  • npm run lint (autofix, no changes)
  • npm test — full gulp suite, 13 tasks, all targets (ts, src, apache-arrow; es5/es2015/esnext across cjs, esm, umd)

Related

@domoritz
domoritz merged commit 5eb0f09 into apache:main Sep 11, 2026
14 checks passed
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.

2 participants