Fix translation for compare_scans message#1480
Fix translation for compare_scans message#1480Vineshnayak wants to merge 1 commit intoOWASP:masterfrom
Conversation
Signed-off-by: Vinesh nayak <vineshnayak.jatothu@gmail.com>
Summary by CodeRabbit
WalkthroughUpdated Hindi localization text for the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@nettacker/locale/hi.yaml`:
- Around line 120-125: The commits that modified the locale keys (compare_scans,
compare_report_path_filename, no_scan_to_compare, compare_report_saved,
build_compare_report, finish_build_report) are unsigned and must be GPG-signed
before merging; fix this by signing the commit(s) (e.g., run git commit --amend
-S for the latest commit or use git rebase --exec 'git commit --amend -S
--no-edit' origin/master to sign multiple commits), then push the rewritten
history with git push --force-with-lease so the pre-merge checks pass.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: a3229f10-43b3-4228-90dc-edcf6e2179e2
📒 Files selected for processing (1)
nettacker/locale/hi.yaml
| compare_scans: "scan_id का उपयोग करके वर्तमान स्कैन की तुलना पुराने स्कैन से करें" | ||
| compare_report_path_filename: तुलना रिपोर्ट सहेजने का फ़ाइल पथ | ||
| no_scan_to_compare: तुलना करने के लिए scan_id नहीं मिला | ||
| compare_report_saved: "{0} में तुलना परिणाम सहेजे गए" | ||
| build_compare_report: "तुलना रिपोर्ट बनाई जा रही है" | ||
| finish_build_report: "तुलना रिपोर्ट तैयार हो गई" No newline at end of file | ||
| finish_build_report: "तुलना रिपोर्ट तैयार हो गई" |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify signature status for commits in this PR branch vs master
git log --pretty=format:'%h %G? %an %s' origin/master..HEAD
# Expected: signed commits (typically G/U depending on trust state).
# Investigate any N/B/E/X/Y/R statuses as unsigned/invalid signatures.Repository: OWASP/Nettacker
Length of output: 189
All commits in this PR must be GPG signed before merge.
@Vineshnayak: The commit in this PR is currently unsigned. This PR will fail pre-merge checks and cannot be merged until you sign the commit(s).
To fix this:
- Sign the commit:
git commit --amend -S - Force-push the updated history:
git push --force-with-lease
Alternatively, if you have multiple unsigned commits, you can use an interactive rebase to sign them all:
git rebase --exec 'git commit --amend -S --no-edit' origin/master
git push --force-with-lease
Signed commits are a mandatory requirement in this repository.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@nettacker/locale/hi.yaml` around lines 120 - 125, The commits that modified
the locale keys (compare_scans, compare_report_path_filename,
no_scan_to_compare, compare_report_saved, build_compare_report,
finish_build_report) are unsigned and must be GPG-signed before merging; fix
this by signing the commit(s) (e.g., run git commit --amend -S for the latest
commit or use git rebase --exec 'git commit --amend -S --no-edit' origin/master
to sign multiple commits), then push the rewritten history with git push
--force-with-lease so the pre-merge checks pass.
Proposed change
Improved Hindi translations for scan comparison-related messages and fixed minor grammar issues.
Type of change
Checklist