Skip to content

Fix EOL MySQL image, remove dead test helper, gitignore .npmrc#145

Merged
jgowdy-godaddy merged 2 commits intomainfrom
bugfixes-round2
Mar 4, 2026
Merged

Fix EOL MySQL image, remove dead test helper, gitignore .npmrc#145
jgowdy-godaddy merged 2 commits intomainfrom
bugfixes-round2

Conversation

@jgowdy-godaddy
Copy link
Collaborator

Summary

  • mysql:5.7 -> mysql:latest: MySQL 5.7 reached EOL October 2023. Updated CI workflow, local-integration-test.sh, and integration/node/run-local.sh. Also dropped unnecessary --platform linux/amd64 from local scripts since mysql:latest has native ARM images.
  • Remove unused get_sample_json: Never called anywhere; also had a broken JSON.stringify replacer (['nested.secret'] doesn't work as a dotted path). Removed along with the now-unused crypto import.
  • Add .npmrc to .gitignore: publish.yml creates a temporary .npmrc with the NPM auth token. Gitignoring it prevents accidental commit of credentials.

Test plan

  • All 53 tests pass
  • ESLint clean on modified files

jgowdy added 2 commits March 4, 2026 11:08
MySQL 5.7 reached end of life in October 2023. Switch all references
to mysql:latest so we automatically pick up new releases. Also drop
--platform linux/amd64 from local scripts since mysql:latest provides
native ARM images.
get_sample_json was never called and had a broken JSON.stringify
replacer. Also gitignore .npmrc to prevent accidental commit of the
temporary auth token file created during CI publish.
@jgowdy-godaddy jgowdy-godaddy merged commit a3a321c into main Mar 4, 2026
2 checks passed
@jgowdy-godaddy jgowdy-godaddy deleted the bugfixes-round2 branch March 4, 2026 19:18
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