Skip to content

Remove Cursor translation rules#1408

Merged
janrysavy merged 1 commit into
OpenSalamander:mainfrom
janrysavy:codex/remove-cursor-rules
May 31, 2026
Merged

Remove Cursor translation rules#1408
janrysavy merged 1 commit into
OpenSalamander:mainfrom
janrysavy:codex/remove-cursor-rules

Conversation

@janrysavy

Copy link
Copy Markdown
Contributor

Summary

  • remove the repository-local Cursor rule file for comment translation work
  • leave the rest of the repository unchanged

Validation

  • git diff --cached --check before commit
  • verified the commit deletes only .cursor/rules/comments-translation.mdc

@janrysavy janrysavy marked this pull request as ready for review May 31, 2026 15:48
@janrysavy janrysavy merged commit 91c9c7b into OpenSalamander:main May 31, 2026
10 checks passed
0xeb added a commit to 0xeb/sally that referenced this pull request Jun 29, 2026
Record reviewed upstream ancestry with -s ours so GitHub reports 0 behind.

Merge-base: 825f8cb
Upstream head: 86a183f
Divergence before: fork-only 706, upstream-only 6
Substance hunt: files-touched 449; comment-only 406; CODE-CHANGED 11; ADDED 0; DELETED 0; RAW 32

Adopted: none.
Skipped:
- 406 source files identical after comment stripping/whitespace normalization.
- 11 CODE-CHANGED files only translate TRACE/message text or fix English wording; no behavior change.
- 32 RAW files are workflow/readme/help/CSS/text cleanup tied to upstream comment-translation maintenance and not applicable to Sally.

Upstream-only commits reviewed:
- fcd8322 Fix translated comment wording (OpenSalamander#1406)
- b95c9ce Translate Czech trace messages (OpenSalamander#1407)
- 91c9c7b Remove Cursor translation rules (OpenSalamander#1408)
- c1023fa Remove comment translation workflows (OpenSalamander#1409)
- 73cd017 Remove comment translation markers (OpenSalamander#1411)
- 86a183f Remove outdated Czech comment translation note from README (OpenSalamander#1413)
@janrysavy janrysavy deleted the codex/remove-cursor-rules branch July 3, 2026 10:48
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.

1 participant