Commit 07481eb
committed
fix(sensitive-issue-searcher): modernise for PHP 8.4
The tool's dependencies were pinned to knplabs/github-api ^2.1 and
cache/redis-adapter ^0.5, both requiring PHP ^5.6 || ^7.0, so composer
install failed on PHP 8.4. The new php-test workflow surfaced this.
- Bump to knplabs/github-api ^3.16 with guzzlehttp/guzzle ^7.13 and
http-interop/http-factory-guzzle, matching the other tools.
- Drop cache/redis-adapter: it was declared but never used by search.php.
- Adapt search.php to the github-api v3 API:
- authenticate() no longer takes Client::AUTH_HTTP_TOKEN; use
AuthMethod::ACCESS_TOKEN.
- Issue::setPerPage() was removed; pass the page size to ResultPager
instead.
- Stop tracking vendor/ (it was committed here unlike every other tool;
.gitignore already ignores it).
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>1 parent 7dd0c3e commit 07481eb
469 files changed
Lines changed: 525 additions & 45286 deletions
File tree
- sensitive-issue-searcher
- vendor
- cache
- adapter-common
- .github
- Exception
- Tests
- hierarchical-cache
- .github
- Tests
- Helper
- redis-adapter
- .github
- Tests
- tag-interop
- .github
- clue/stream-filter
- src
- tests
- composer
- guzzlehttp
- guzzle
- src
- Cookie
- Exception
- Handler
- promises
- src
- psr7
- src
- knplabs/github-api
- lib/Github
- Api
- CurrentUser
- Enterprise
- Gist
- GitData
- Issue
- Organization
- Project
- PullRequest
- Repository
- Exception
- HttpClient
- Message
- Plugin
- php-http
- cache-plugin
- src
- client-common
- src
- Exception
- Plugin
- discovery
- src
- Exception
- Strategy
- guzzle6-adapter
- src
- httplug
- src
- Exception
- Promise
- message-factory
- src
- message
- src
- Authentication
- Builder
- Decorator
- Encoding
- Filter
- Formatter
- MessageFactory
- RequestMatcher
- StreamFactory
- Stream
- UriFactory
- promise
- src
- psr
- cache
- src
- http-message
- src
- log
- Psr/Log
- Test
- simple-cache
- src
- symfony/options-resolver
- Exception
- Tests
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
4 | | - | |
5 | | - | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
0 commit comments