Skip to content

chore: remove scratch logs committed by mistake in #123 - #124

Merged
moshest merged 1 commit into
mainfrom
claude/remove-stray-logs
Aug 31, 2026
Merged

chore: remove scratch logs committed by mistake in #123#124
moshest merged 1 commit into
mainfrom
claude/remove-stray-logs

Conversation

@moshest

@moshest moshest commented Aug 27, 2026

Copy link
Copy Markdown
Member

Summary

node22.log (8 KB) and node24.log (24 KB) are local verification output from testing the Node bump. They were committed by mistake in #123.

How it slipped through: I reviewed that diff with git diff --stat before git add -A. git diff only reports tracked modifications, so two untracked log files were invisible at review time and then swept in by add -A.

Adds *.log to .gitignore so the same slip can't repeat.

Test plan

Deletes two untracked-by-intent files and adds a .gitignore line. No code, config, or workflow touched — nothing to test beyond CI staying green.


Generated by Claude Code

node22.log and node24.log are local verification output from testing the
Node bump. They were never meant to be committed — `git add -A` swept them
up because the diff was reviewed with `git diff --stat` beforehand, which
only shows tracked changes and so never displayed them.

Adds *.log to .gitignore so the same slip cannot repeat.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NBQQpA86yYzwJUiVz8ph2R
@changeset-bot

changeset-bot Bot commented Aug 27, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 54c4fb5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@moshest
moshest marked this pull request as ready for review August 31, 2026 16:42
@moshest
moshest merged commit c331648 into main Aug 31, 2026
3 checks passed
@moshest
moshest deleted the claude/remove-stray-logs branch August 31, 2026 16:42
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