chore: maintenance pass – deps, CI modernization, repo-checker fixes - #376
Merged
Conversation
- engines.node -> >=22; drop EOL Node 20 (W0028) - @iobroker/adapter-core ^3.3.2 -> ^3.4.3 (W0034) - @types/node ^25 -> ^22.10.0 to match Node 22 typings (W0066) - add @iobroker/adapter-dev, @tsconfig/node22; tsconfig extends node22 (S0062/S0085/S0088) - remove gulpfile.js, rely on adapter-dev translate (W5013) - test-and-release.yml: node 24 for check-and-lint/deploy, testing-action-check@v2, adapter-tests needs check-and-lint + matrix node [22,24] x [ubuntu,windows,macos], fail-fast:false, timeout 30, guarded concurrency group (E3020/E3022/E3027/W3027/S3014) - io-package.json: admin globalDependency >=7.6.20 (W1056) - dependabot.yml: npm cooldown 7d, ignore @types/node major & typescript minor+major, limit 15 (W8915/W8917) - remove dependabot-auto-merge.yml.OLD (S8910) - README: add Sentry notice (W6023), CHANGELOG_OLD link (S6022), maintenance changelog entry - admin/i18n/en: add "Seconds between data polling" key (W5612) - untrack committed .DS_Store files, add to .gitignore - release script --branchPattern Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Maintenance pass addressing the ioBroker repository checker findings (issue #365), modernising CI/release tooling to the org standard, and refreshing dependencies. No adapter source code was changed — only config, CI, docs and dependencies.
Checker findings fixed
Errors
E3020/E3022—test-and-release.yml:check-and-lintanddeploynow use Node.js 24.E3027—adapter-testsmatrix now covers windows + macos (node 22 & 24).Warnings
W0028—engines.noderaised to>=22(drops EOL Node 20).W0034—@iobroker/adapter-core^3.3.2→^3.4.3.W0066—@types/node^25→^22.10.0(matches Node 22 typings).W1056— io-packageglobalDependencies.admin→>=7.6.20.W3027— adapter-tests OS matrix (see E3027).W5013— removedgulpfile.js; translation flow relies on@iobroker/adapter-dev.W5612— added missingSeconds between data pollingkey toadmin/i18n/en.W6023— added the standard Sentry notice toREADME.md.W8915— dependabot npmcooldown: { default-days: 7 }.W8917— dependabot ignore rule for@types/nodesemver-major (plus typescript minor+major); limit 15.Suggestions
S0062/S0085/S0088— added@iobroker/adapter-devand@tsconfig/node22;tsconfig.jsonnow extends@tsconfig/node22/tsconfig.json.S3014—adapter-testsdeclaresneeds: [check-and-lint].S6022— README links toCHANGELOG_OLD.md.S8910— removed.github/workflows/dependabot-auto-merge.yml.OLD.Also: guarded concurrency group (
${{ github.workflow }}-${{ github.ref }}, cancel-in-progress skipped for tags),fail-fast: false,timeout-minutes: 30; untracked committed.DS_Storefiles and added them to.gitignore;releasescript gains--branchPattern '*'.Bot PRs absorbed by this PR
These iobroker-bot / dependabot PRs are superseded and can be closed once this merges:
Leftovers (not addressed here)
S5601—admin/i18nstill uses the long format ({lang}/translations.json); migrating to the short format is a separate, riskier change.S5005— plainsetTimeout()inmain.js(module-leveltimerwith explicitclearTimeout); converting tothis.setTimeout()is optional and left as-is to avoid changing runtime behaviour.W3050/W3052/S3056— informational: checker could not retrieve logs for old (>90 day) workflow runs; resolves naturally on the next release.Local verification
npm run lint— pass (0 errors; only pre-existing jsdoc warnings)npm run test:package— pass (57 passing)npm run test:integration— pass (adapter boots on Node 24 / js-controller 7.2.3, "1 passing"). Note: the run initially collided with a shared test port (19001) because several sibling repos were running integration tests concurrently on the same machine; it passed cleanly once run in an isolated window.🤖 Generated with Claude Code