Skip to content

Use Crowdin GitHub Action#60

Merged
ryanlua merged 11 commits intomainfrom
crowdin
Apr 22, 2025
Merged

Use Crowdin GitHub Action#60
ryanlua merged 11 commits intomainfrom
crowdin

Conversation

@ryanlua
Copy link
Owner

@ryanlua ryanlua commented Apr 22, 2025

Description

Use the Crowdin GitHub Action instead of the native GitHub integration

Related Issues

Changes Made

  • Add new GitHub Action for Crowdin
  • Add Crowdin to issue templates
  • Update missing localization

Checklist

  • I have tested these changes thoroughly.
  • I have reviewed my code for any potential errors or issues.
  • I have followed the code style guidelines for this project.

Additional Notes

ryanlua added 11 commits April 21, 2025 14:56
Signed-off-by: Ryan Luu <ryan.luu@ryanluu.dev>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Copilot AI review requested due to automatic review settings April 22, 2025 21:41
@ryanlua ryanlua self-assigned this Apr 22, 2025
@github-actions
Copy link

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
actions/actions/checkout 4.*.* 🟢 5.6
Details
CheckScoreReason
Code-Review🟢 10all changesets reviewed
Maintained⚠️ 10 commit(s) and 2 issue activity found in the last 90 days -- score normalized to 1
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Binary-Artifacts🟢 10no binaries found in the repo
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
License🟢 10license file detected
Fuzzing⚠️ 0project is not fuzzed
Packaging⚠️ -1packaging workflow not detected
Signed-Releases⚠️ -1no releases found
Pinned-Dependencies🟢 3dependency not pinned by hash detected -- score normalized to 3
Security-Policy🟢 9security policy file detected
Branch-Protection⚠️ -1internal error: error during GetBranch(releases/v2): error during branchesHandler.query: internal error: githubv4.Query: Resource not accessible by integration
SAST🟢 9SAST tool detected but not run on all commits
Vulnerabilities⚠️ 28 existing vulnerabilities detected
actions/crowdin/github-action 2.*.* 🟢 4.9
Details
CheckScoreReason
Packaging⚠️ -1packaging workflow not detected
Code-Review⚠️ 1Found 3/30 approved changesets -- score normalized to 1
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Binary-Artifacts🟢 10no binaries found in the repo
Maintained🟢 109 commit(s) and 6 issue activity found in the last 90 days -- score normalized to 10
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Vulnerabilities🟢 100 existing vulnerabilities detected
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Fuzzing⚠️ 0project is not fuzzed
Security-Policy⚠️ 0security policy file not detected
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: githubv4.Query: Resource not accessible by integration
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0

Scanned Files

  • .github/workflows/crowdin.yml

Copy link
Contributor

Copilot AI left a 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 pull request introduces a Crowdin GitHub Action to replace the native GitHub integration, updates issue templates, and adjusts localization file paths.

  • Added configuration for the Crowdin GitHub Action in workflow files and a new root-level configuration file.
  • Updated issue templates and a settings file to incorporate localization and system information changes.

Reviewed Changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
crowdin.yml Added environment variables and updated file path case sensitivity for resources.
FluentAutoClicker/SettingsPage.xaml.cs Replaced placeholder feedback text with system information details.
.github/workflows/crowdin.yml Introduced a new workflow to synchronize with Crowdin using the latest action version.
.github/ISSUE_TEMPLATE/feature_request.yml Minor update to the description field formatting.
.github/ISSUE_TEMPLATE/config.yml Revised contact link entries and adjusted wording in the about sections.
.github/ISSUE_TEMPLATE/bug_report.yml Updated description text formatting.
Files not reviewed (2)
  • FluentAutoClicker/SettingsPage.xaml: Language not supported
  • FluentAutoClicker/Strings/en-US/Resources.resw: Language not supported
Comments suppressed due to low confidence (2)

.github/ISSUE_TEMPLATE/config.yml:6

  • [nitpick] Consider capitalizing the name consistently (e.g., 'Get Community Support') to match the style used in other entries.
  - name: Get community support

crowdin.yml:7

  • The resource file path was changed from 'en-us' to 'en-US'. Please confirm that the updated case matches the actual directory structure to avoid file not found errors.
  - source: /FluentAutoClicker/Strings/en-US/Resources.resw

@ryanlua ryanlua merged commit 9c25b0e into main Apr 22, 2025
5 checks passed
@ryanlua ryanlua deleted the crowdin branch April 22, 2025 21:57
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