2024-12-21 23:40:53 +01:00
|
|
|
name: Check Properties Files on PR
|
2024-08-25 23:04:28 +02:00
|
|
|
|
|
|
|
on:
|
|
|
|
pull_request_target:
|
|
|
|
types: [opened, synchronize, reopened]
|
|
|
|
paths:
|
|
|
|
- "src/main/resources/messages_*.properties"
|
2024-12-21 23:40:53 +01:00
|
|
|
|
2024-12-29 15:44:50 +01:00
|
|
|
permissions:
|
|
|
|
contents: read # Allow read access to repository content
|
2024-08-31 15:54:11 +02:00
|
|
|
|
2024-08-25 23:04:28 +02:00
|
|
|
jobs:
|
|
|
|
check-files:
|
2024-08-31 15:54:11 +02:00
|
|
|
if: github.event_name == 'pull_request_target'
|
2024-08-25 23:04:28 +02:00
|
|
|
runs-on: ubuntu-latest
|
2025-01-02 16:58:08 +01:00
|
|
|
permissions:
|
|
|
|
issues: write # Allow posting comments on issues/PRs
|
2025-01-02 18:28:32 +01:00
|
|
|
pull-requests: write
|
2024-08-25 23:04:28 +02:00
|
|
|
steps:
|
2024-12-21 13:28:35 +01:00
|
|
|
- name: Harden Runner
|
|
|
|
uses: step-security/harden-runner@0080882f6c36860b6ba35c610c98ce87d4e2f26f # v2.10.2
|
|
|
|
with:
|
|
|
|
egress-policy: audit
|
|
|
|
|
2024-11-20 10:44:51 +01:00
|
|
|
- name: Checkout main branch first
|
2024-12-21 13:28:35 +01:00
|
|
|
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
|
2024-08-25 23:04:28 +02:00
|
|
|
|
|
|
|
- name: Set up Python
|
2024-12-21 13:28:35 +01:00
|
|
|
uses: actions/setup-python@0b93645e9fea7318ecaed2b359559ac225c90a2b # v5.3.0
|
2024-08-25 23:04:28 +02:00
|
|
|
with:
|
2025-01-08 16:33:35 +01:00
|
|
|
python-version: "3.12"
|
2024-08-25 23:04:28 +02:00
|
|
|
|
2025-01-02 16:58:08 +01:00
|
|
|
- name: Get PR data
|
|
|
|
id: get-pr-data
|
|
|
|
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
|
|
|
|
with:
|
|
|
|
script: |
|
|
|
|
const prNumber = context.payload.pull_request.number;
|
|
|
|
const repoOwner = context.payload.repository.owner.login;
|
|
|
|
const repoName = context.payload.repository.name;
|
|
|
|
const branch = context.payload.pull_request.head.ref;
|
|
|
|
|
|
|
|
console.log(`PR Number: ${prNumber}`);
|
|
|
|
console.log(`Repo Owner: ${repoOwner}`);
|
|
|
|
console.log(`Repo Name: ${repoName}`);
|
|
|
|
console.log(`Branch: ${branch}`);
|
|
|
|
|
|
|
|
core.setOutput("pr_number", prNumber);
|
|
|
|
core.setOutput("repo_owner", repoOwner);
|
|
|
|
core.setOutput("repo_name", repoName);
|
|
|
|
core.setOutput("branch", branch);
|
|
|
|
continue-on-error: true
|
2025-01-02 18:28:32 +01:00
|
|
|
|
|
|
|
- name: Fetch PR changed files
|
|
|
|
id: fetch-pr-changes
|
|
|
|
env:
|
|
|
|
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
|
|
|
run: |
|
|
|
|
echo "Fetching PR changed files..."
|
|
|
|
echo "Getting list of changed files from PR..."
|
|
|
|
gh pr view ${{ steps.get-pr-data.outputs.pr_number }} --json files -q ".files[].path" | grep -E '^src/main/resources/messages_[a-zA-Z_]+\.properties$' > changed_files.txt # Filter only matching property files
|
|
|
|
|
2025-01-02 16:58:08 +01:00
|
|
|
- name: Determine reference file test
|
2025-01-02 18:28:32 +01:00
|
|
|
id: determine-file
|
2025-01-02 16:58:08 +01:00
|
|
|
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
|
|
|
|
with:
|
|
|
|
script: |
|
2025-01-02 18:28:32 +01:00
|
|
|
const fs = require("fs");
|
|
|
|
const path = require("path");
|
|
|
|
|
2025-01-02 16:58:08 +01:00
|
|
|
const prNumber = ${{ steps.get-pr-data.outputs.pr_number }};
|
|
|
|
const repoOwner = "${{ steps.get-pr-data.outputs.repo_owner }}";
|
|
|
|
const repoName = "${{ steps.get-pr-data.outputs.repo_name }}";
|
2025-01-02 18:28:32 +01:00
|
|
|
|
|
|
|
const prRepoOwner = "${{ github.event.pull_request.head.repo.owner.login }}";
|
|
|
|
const prRepoName = "${{ github.event.pull_request.head.repo.name }}";
|
2025-01-02 16:58:08 +01:00
|
|
|
const branch = "${{ steps.get-pr-data.outputs.branch }}";
|
|
|
|
|
|
|
|
console.log(`Determining reference file for PR #${prNumber}`);
|
|
|
|
|
|
|
|
// Validate inputs
|
|
|
|
const validateInput = (input, regex, name) => {
|
|
|
|
if (!regex.test(input)) {
|
|
|
|
throw new Error(`Invalid ${name}: ${input}`);
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
validateInput(repoOwner, /^[a-zA-Z0-9_-]+$/, "repository owner");
|
|
|
|
validateInput(repoName, /^[a-zA-Z0-9._-]+$/, "repository name");
|
|
|
|
validateInput(branch, /^[a-zA-Z0-9._/-]+$/, "branch name");
|
|
|
|
|
|
|
|
// Get the list of changed files in the PR
|
|
|
|
const { data: files } = await github.rest.pulls.listFiles({
|
|
|
|
owner: repoOwner,
|
|
|
|
repo: repoName,
|
|
|
|
pull_number: prNumber,
|
|
|
|
});
|
|
|
|
|
|
|
|
// Filter for relevant files based on the PR changes
|
|
|
|
const changedFiles = files
|
|
|
|
.map(file => file.filename)
|
|
|
|
.filter(file => /^src\/main\/resources\/messages_[a-zA-Z_]+\.properties$/.test(file));
|
|
|
|
|
|
|
|
console.log("Changed files:", changedFiles);
|
|
|
|
|
|
|
|
// Create a temporary directory for PR files
|
|
|
|
const tempDir = "pr-branch";
|
2025-01-02 18:28:32 +01:00
|
|
|
if (!fs.existsSync(tempDir)) {
|
|
|
|
fs.mkdirSync(tempDir, { recursive: true });
|
2025-01-02 16:58:08 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
// Download and save each changed file
|
|
|
|
for (const file of changedFiles) {
|
|
|
|
const { data: fileContent } = await github.rest.repos.getContent({
|
2025-01-02 18:28:32 +01:00
|
|
|
owner: prRepoOwner,
|
|
|
|
repo: prRepoName,
|
2025-01-02 16:58:08 +01:00
|
|
|
path: file,
|
|
|
|
ref: branch,
|
|
|
|
});
|
|
|
|
|
|
|
|
const content = Buffer.from(fileContent.content, "base64").toString("utf-8");
|
2025-01-02 18:28:32 +01:00
|
|
|
const filePath = path.join(tempDir, file);
|
|
|
|
const dirPath = path.dirname(filePath);
|
2025-01-02 16:58:08 +01:00
|
|
|
|
2025-01-02 18:28:32 +01:00
|
|
|
if (!fs.existsSync(dirPath)) {
|
|
|
|
fs.mkdirSync(dirPath, { recursive: true });
|
2025-01-02 16:58:08 +01:00
|
|
|
}
|
|
|
|
|
2025-01-02 18:28:32 +01:00
|
|
|
fs.writeFileSync(filePath, content);
|
2025-01-02 16:58:08 +01:00
|
|
|
console.log(`Saved file: ${filePath}`);
|
|
|
|
}
|
|
|
|
|
|
|
|
// Output the list of changed files for further processing
|
|
|
|
const fileList = changedFiles.join(" ");
|
|
|
|
core.exportVariable("FILES_LIST", fileList);
|
|
|
|
console.log("Files saved and listed in FILES_LIST.");
|
|
|
|
|
|
|
|
// Determine reference file
|
|
|
|
let referenceFilePath;
|
|
|
|
if (changedFiles.includes("src/main/resources/messages_en_GB.properties")) {
|
|
|
|
console.log("Using PR branch reference file.");
|
|
|
|
const { data: fileContent } = await github.rest.repos.getContent({
|
2025-01-02 18:28:32 +01:00
|
|
|
owner: prRepoOwner,
|
|
|
|
repo: prRepoName,
|
2025-01-02 16:58:08 +01:00
|
|
|
path: "src/main/resources/messages_en_GB.properties",
|
|
|
|
ref: branch,
|
|
|
|
});
|
|
|
|
|
|
|
|
referenceFilePath = "pr-branch-messages_en_GB.properties";
|
|
|
|
const content = Buffer.from(fileContent.content, "base64").toString("utf-8");
|
2025-01-02 18:28:32 +01:00
|
|
|
fs.writeFileSync(referenceFilePath, content);
|
2025-01-02 16:58:08 +01:00
|
|
|
} else {
|
|
|
|
console.log("Using main branch reference file.");
|
|
|
|
const { data: fileContent } = await github.rest.repos.getContent({
|
2025-01-02 18:28:32 +01:00
|
|
|
owner: repoOwner,
|
|
|
|
repo: repoName,
|
2025-01-02 16:58:08 +01:00
|
|
|
path: "src/main/resources/messages_en_GB.properties",
|
|
|
|
ref: "main",
|
|
|
|
});
|
|
|
|
|
|
|
|
referenceFilePath = "main-branch-messages_en_GB.properties";
|
|
|
|
const content = Buffer.from(fileContent.content, "base64").toString("utf-8");
|
2025-01-02 18:28:32 +01:00
|
|
|
fs.writeFileSync(referenceFilePath, content);
|
2025-01-02 16:58:08 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
console.log(`Reference file path: ${referenceFilePath}`);
|
|
|
|
core.exportVariable("REFERENCE_FILE", referenceFilePath);
|
2024-08-25 23:04:28 +02:00
|
|
|
|
|
|
|
- name: Run Python script to check files
|
|
|
|
id: run-check
|
|
|
|
run: |
|
2024-11-20 10:44:51 +01:00
|
|
|
echo "Running Python script to check files..."
|
2024-12-29 15:44:50 +01:00
|
|
|
python .github/scripts/check_language_properties.py \
|
2024-11-21 12:31:32 +01:00
|
|
|
--actor ${{ github.event.pull_request.user.login }} \
|
2024-11-20 10:44:51 +01:00
|
|
|
--reference-file "${REFERENCE_FILE}" \
|
2024-12-29 15:44:50 +01:00
|
|
|
--branch "pr-branch" \
|
2025-01-02 16:58:08 +01:00
|
|
|
--files "${FILES_LIST[@]}" > result.txt
|
|
|
|
continue-on-error: true # Continue the job even if this step fails
|
2024-08-25 23:04:28 +02:00
|
|
|
|
|
|
|
- name: Capture output
|
|
|
|
id: capture-output
|
|
|
|
run: |
|
2024-11-23 12:49:49 +01:00
|
|
|
if [ -f result.txt ] && [ -s result.txt ]; then
|
|
|
|
echo "Test, capturing output..."
|
|
|
|
SCRIPT_OUTPUT=$(cat result.txt)
|
|
|
|
echo "SCRIPT_OUTPUT<<EOF" >> $GITHUB_ENV
|
|
|
|
echo "$SCRIPT_OUTPUT" >> $GITHUB_ENV
|
2024-08-25 23:04:28 +02:00
|
|
|
echo "EOF" >> $GITHUB_ENV
|
2024-11-23 12:49:49 +01:00
|
|
|
echo "${SCRIPT_OUTPUT}"
|
|
|
|
|
2024-12-29 15:44:50 +01:00
|
|
|
# Determine job failure based on script output
|
2024-11-23 12:49:49 +01:00
|
|
|
if [[ "$SCRIPT_OUTPUT" == *"❌"* ]]; then
|
|
|
|
echo "FAIL_JOB=true" >> $GITHUB_ENV
|
|
|
|
else
|
|
|
|
echo "FAIL_JOB=false" >> $GITHUB_ENV
|
|
|
|
fi
|
2024-08-31 15:54:11 +02:00
|
|
|
else
|
2024-11-23 12:49:49 +01:00
|
|
|
echo "No update found."
|
|
|
|
echo "SCRIPT_OUTPUT=" >> $GITHUB_ENV
|
|
|
|
echo "FAIL_JOB=false" >> $GITHUB_ENV
|
2024-08-25 23:04:28 +02:00
|
|
|
fi
|
|
|
|
|
|
|
|
- name: Post comment on PR
|
2024-11-23 12:49:49 +01:00
|
|
|
if: env.SCRIPT_OUTPUT != ''
|
2024-12-21 13:28:35 +01:00
|
|
|
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
|
2024-08-25 23:04:28 +02:00
|
|
|
with:
|
|
|
|
script: |
|
2024-11-23 12:49:49 +01:00
|
|
|
const { GITHUB_REPOSITORY, SCRIPT_OUTPUT } = process.env;
|
2024-08-25 23:04:28 +02:00
|
|
|
const [repoOwner, repoName] = GITHUB_REPOSITORY.split('/');
|
2025-01-02 18:28:32 +01:00
|
|
|
const issueNumber = context.issue.number;
|
2024-08-31 15:54:11 +02:00
|
|
|
|
|
|
|
// Find existing comment
|
|
|
|
const comments = await github.rest.issues.listComments({
|
2024-08-25 23:04:28 +02:00
|
|
|
owner: repoOwner,
|
|
|
|
repo: repoName,
|
2025-01-02 18:28:32 +01:00
|
|
|
issue_number: issueNumber
|
2024-08-25 23:04:28 +02:00
|
|
|
});
|
2024-08-31 15:54:11 +02:00
|
|
|
|
|
|
|
const comment = comments.data.find(c => c.body.includes("## 🚀 Translation Verification Summary"));
|
2024-11-21 12:31:32 +01:00
|
|
|
|
2024-12-29 15:44:50 +01:00
|
|
|
// Only update or create comments by the action user
|
2024-08-31 15:54:11 +02:00
|
|
|
const expectedActor = "github-actions[bot]";
|
|
|
|
|
|
|
|
if (comment && comment.user.login === expectedActor) {
|
|
|
|
// Update existing comment
|
|
|
|
await github.rest.issues.updateComment({
|
|
|
|
owner: repoOwner,
|
|
|
|
repo: repoName,
|
|
|
|
comment_id: comment.id,
|
2024-11-23 12:49:49 +01:00
|
|
|
body: `## 🚀 Translation Verification Summary\n\n\n${SCRIPT_OUTPUT}\n`
|
2024-08-31 15:54:11 +02:00
|
|
|
});
|
|
|
|
console.log("Updated existing comment.");
|
|
|
|
} else if (!comment) {
|
|
|
|
// Create new comment if no existing comment is found
|
|
|
|
await github.rest.issues.createComment({
|
|
|
|
owner: repoOwner,
|
|
|
|
repo: repoName,
|
2025-01-02 18:28:32 +01:00
|
|
|
issue_number: issueNumber,
|
2024-11-23 12:49:49 +01:00
|
|
|
body: `## 🚀 Translation Verification Summary\n\n\n${SCRIPT_OUTPUT}\n`
|
2024-08-31 15:54:11 +02:00
|
|
|
});
|
|
|
|
console.log("Created new comment.");
|
|
|
|
} else {
|
|
|
|
console.log("Comment update attempt denied. Actor does not match.");
|
|
|
|
}
|
|
|
|
|
2024-11-23 12:49:49 +01:00
|
|
|
- name: Fail job if errors found
|
|
|
|
if: env.FAIL_JOB == 'true'
|
|
|
|
run: |
|
|
|
|
echo "Failing the job because errors were detected."
|
|
|
|
exit 1
|