Skip to content

feat: upgrade minimatch dependency to 10.2.1#545

Merged
hsluoyz merged 5 commits intomasterfrom
copilot/upgrade-minimatch-dependency
Feb 19, 2026
Merged

feat: upgrade minimatch dependency to 10.2.1#545
hsluoyz merged 5 commits intomasterfrom
copilot/upgrade-minimatch-dependency

Conversation

Copy link
Contributor

Copilot AI commented Feb 19, 2026

fix: #544

Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI and others added 2 commits February 19, 2026 12:59
…fixes

Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
Copilot AI changed the title [WIP] Upgrade minimatch dependency to version 10.2.1 Upgrade minimatch to 10.2.1 Feb 19, 2026
Copilot AI requested a review from hsluoyz February 19, 2026 13:03
@hsluoyz hsluoyz marked this pull request as ready for review February 19, 2026 13:04
@hsluoyz
Copy link
Member

hsluoyz commented Feb 19, 2026

@copilot fix:

Run yarn install
  yarn install
  shell: /usr/bin/bash -e {0}
yarn install v1.22.22
error SyntaxError: Invalid value type 1081:0 in /home/runner/work/node-casbin/node-casbin/yarn.lock
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
    at Parser.unexpected (/usr/local/lib/node_modules/yarn/lib/cli.js:64026:11)
    at Parser.parse (/usr/local/lib/node_modules/yarn/lib/cli.js:64154:16)
    at Parser.parse (/usr/local/lib/node_modules/yarn/lib/cli.js:64131:26)
    at parse (/usr/local/lib/node_modules/yarn/lib/cli.js:64231:21)
    at module.exports.exports.default (/usr/local/lib/node_modules/yarn/lib/cli.js:63793:96)
    at Function.<anonymous> (/usr/local/lib/node_modules/yarn/lib/cli.js:3068:63)
    at Generator.next (<anonymous>)
    at step (/usr/local/lib/node_modules/yarn/lib/cli.js:310:30)
    at /usr/local/lib/node_modules/yarn/lib/cli.js:321:13
Error: Process completed with exit code 1.

Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 19, 2026

@copilot fix:

...

Fixed in commit 988decf. The issue was caused by using npm install which corrupted the yarn.lock file. I've regenerated it properly using yarn install and also fixed an additional TypeScript type narrowing issue in the test file. All tests and builds now pass.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 22183354721

Details

  • 7 of 11 (63.64%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.09%) to 78.701%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/util/util.ts 0 4 0.0%
Totals Coverage Status
Change from base Build 20878530147: 0.09%
Covered Lines: 1711
Relevant Lines: 2069

💛 - Coveralls

@hsluoyz hsluoyz changed the title Upgrade minimatch to 10.2.1 feat: upgrade minimatch dependency to 10.2.1 Feb 19, 2026
@hsluoyz hsluoyz merged commit e5c3004 into master Feb 19, 2026
7 of 9 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.

[feature] upgrade minimatch dependency to latest 10.2.1

4 participants