-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
tr.cursorrules #123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tr.cursorrules #123
Conversation
WalkthroughThe file Changes
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR replaces the English Conventional Commit Messages specification with a full Turkish translation in the .cursorrules
file.
- Translated all commit message guidelines into Turkish
- Replaced English list items and specification details
- Updated keyword definitions and formatted structural markers
Comments suppressed due to low confidence (2)
rules/git-conventional-commit-messages/.cursorrules:1
- List items in this section should be prefixed with a '-' to match the bullet style used elsewhere in the file.
Konvansiyonel Taahhüt Mesajları spesifikasyonunu kullanarak taahhüt mesajları oluşturun
rules/git-conventional-commit-messages/.cursorrules:18
- The phrase 'BREAKING CHANGE:' is duplicated. Remove the extra prefix for clarity.
- BREAKING CHANGE: BREAKING CHANGE: altbilgisi olan veya ! ekleyen bir taahhüt tür/kapsamdan sonra, bir API değişikliğini (Anlamsal Sürümlemedeki MAJOR ile ilişkili) bozan bir API değişikliğini tanıtıyor. BREAKING CHANGE, herhangi bir türdeki commit'lerin bir parçası olabilir.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🔭 Outside diff range comments (1)
rules/git-conventional-commit-messages/.cursorrules (1)
1-42
: Configuration file overwritten with prose – build will break
.cursorrules
is meant to contain Cursor rule definitions (machine-parsable JSON/YAML).
Replacing it with free-form Turkish documentation removes every rule, so any CI step or IDE integration that loads these rules will now fail (or silently stop linting commit messages).Action items
- Restore the original rule file (or recreate an equivalent Turkish-locale rule set in the expected format).
- Move the translated spec to a Markdown file such as
README_tr.md
to keep both documentation and working configuration.If the intention was only to provide docs, rename the file; keeping the
.cursorrules
extension is misleading and hazardous.
🧹 Nitpick comments (2)
rules/git-conventional-commit-messages/.cursorrules (2)
18-18
: Duplicated phrase “BREAKING CHANGE: BREAKING CHANGE:”The repetition looks accidental and makes the sentence hard to read.
- - BREAKING CHANGE: BREAKING CHANGE: altbilgisi olan … + - BREAKING CHANGE: altbilgisi olan …
25-25
: Keyword list repeats “OLMALI”In “ZORUNLU … OLMALI, OLMAMALI, OLMALI”, the second “OLMALI” is duplicated.
- "… OLMALI", "OLMAMALI", "OLMALI", "ÖNERİLEN" … + "… OLMALI", "OLMAMALI", "ÖNERİLEN" …
Summary by CodeRabbit