Jose/ansible_role_proxmox_provision
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore 📦: Update redaction setting in PR check workflow #18

Closed
Jose wants to merge 0 commits from dev into main
pull from: dev
merge into: Jose:main
Conversation 2 Commits - Files Changed - +202 -169
Jose commented 2026-02-14 10:15:02 +01:00
Owner
Copy Link

This commit updates the redaction setting from --redact=10 to --redact=false in the .gitea/workflows/pr-check.yaml file. This change ensures that no redaction occurs during the PR check process, maintaining full visibility and integrity of the data being checked.

This commit updates the redaction setting from `--redact=10` to `--redact=false` in the `.gitea/workflows/pr-check.yaml` file. This change ensures that no redaction occurs during the PR check process, maintaining full visibility and integrity of the data being checked.
Jose added 12 commits 2026-02-14 10:15:02 +01:00
feat ✨: Add PR check workflow for Gitleaks and lint tests
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Details
Gitleaks Scan / gitleaks (push) Successful in 5s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
aad77acf42 
This commit introduces a new pull request (PR) check workflow that includes Gitleaks for security scanning and lint tests to ensure code quality.
refactor ♻️: Refactor PR check workflow by consolidating steps and removing redundant code
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Details
Gitleaks Scan / gitleaks (push) Successful in 5s
Details
Markdown Lint / markdown-lint (push) Successful in 7s
Details
945717ffbb 
This refactoring consolidates the steps in the PR check workflow, reducing redundancy and improving efficiency.
fix 🐛: Fix typo in PR check workflow comments
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Details
Gitleaks Scan / gitleaks (push) Successful in 4s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
1733801fe5 
Corrected a minor spelling error in the comments of the pull request check workflow to improve readability and clarity.
fix 🐛: Fix typo in PR check comment body
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 11s
Details
Gitleaks Scan / gitleaks (push) Successful in 4s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
ai-reviews / Review PR (pull_request) Successful in 19s
Details
PR check / Gitleaks (pull_request) Failing after 5s
Details
PR check / lint tests (pull_request) Successful in 14s
Details
PR check / handle_failures (pull_request) Has been skipped
Details
PR check / handle_success (pull_request) Has been skipped
Details
a0138b1bea 
Corrected a minor spelling error in the comment body of the pull request check to improve readability and accuracy.
refactor ♻️: Refactor condition for handle_failures to run regardless of previous job results
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Details
Gitleaks Scan / gitleaks (push) Successful in 6s
Details
ai-reviews / Review PR (pull_request) Successful in 19s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
PR check / Gitleaks (pull_request) Failing after 5s
Details
PR check / lint tests (pull_request) Failing after 9s
Details
PR check / handle_failures (pull_request) Successful in 2s
Details
PR check / handle_success (pull_request) Has been skipped
Details
dfda760d2c 
This refactoring ensures that the `handle_failures` function is executed irrespective of the outcomes of previous jobs, improving the robustness and reliability of the system.
docs 📝: Update README title and compatibility matrix format
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Details
Gitleaks Scan / gitleaks (push) Successful in 4s
Details
Markdown Lint / markdown-lint (push) Successful in 6s
Details
eb5eaf7887 
Updated the README title to be more descriptive and restructured the compatibility matrix for better readability.
fix 🐛: Fix PR check job condition
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 11s
Details
Gitleaks Scan / gitleaks (push) Successful in 5s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
ai-reviews / Review PR (pull_request) Successful in 21s
Details
PR check / Gitleaks (pull_request) Failing after 6s
Details
PR check / lint tests (pull_request) Successful in 13s
Details
PR check / handle_failures (pull_request) Successful in 1s
Details
PR check / handle_success (pull_request) Has been skipped
Details
fc3b5e8507 
This commit addresses an issue where the PR check job was not running as expected. The condition for triggering the job has been corrected to ensure it runs properly under all circumstances.
docs 📝: Update README.md section title and minor formatting
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 10s
Details
Gitleaks Scan / gitleaks (push) Successful in 4s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
ai-reviews / Review PR (pull_request) Successful in 28s
Details
PR check / Gitleaks (pull_request) Failing after 4s
Details
PR check / lint tests (pull_request) Successful in 13s
Details
PR check / handle_failures (pull_request) Successful in 1s
Details
PR check / handle_success (pull_request) Has been skipped
Details
e21f9bd002 
Corrected the title of a section in the README.md file and made some minor formatting adjustments for better readability.
chore 📦: Update Gitleaks configuration and README
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 12s
Details
Gitleaks Scan / gitleaks (push) Failing after 5s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
c9a6f0e092 
Add `--config-path .gitleaks.toml` to pr-check.yaml and create a new `.gitleaks.toml` file with allowlist and rules for security scanning. Rename 'API utilities' to 'Utilities' in the README.md table.
docs 📝: Comment out allowlist rule for badge URLs in README
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 12s
Details
Gitleaks Scan / gitleaks (push) Successful in 4s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
ai-reviews / Review PR (pull_request) Successful in 30s
Details
PR check / Gitleaks (pull_request) Failing after 4s
Details
PR check / lint tests (pull_request) Successful in 15s
Details
PR check / handle_failures (pull_request) Successful in 2s
Details
PR check / handle_success (pull_request) Has been skipped
Details
dbd70e61ce 
Updated the README to comment out the allowlist rule for badge URLs, as it is no longer necessary.
refactor ♻️: Refactor Gitleaks command option from --config-path to --config
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 12s
Details
Gitleaks Scan / gitleaks (push) Successful in 5s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
ai-reviews / Review PR (pull_request) Successful in 35s
Details
PR check / Gitleaks (pull_request) Failing after 5s
Details
PR check / lint tests (pull_request) Successful in 13s
Details
PR check / handle_failures (pull_request) Successful in 2s
Details
PR check / handle_success (pull_request) Has been skipped
Details
7b5c802689 
Updated the configuration path option for Gitleaks from '--config-path' to '--config' to simplify usage and improve consistency with other tools.
chore 📦: Update redaction setting in PR check workflow
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 12s
Details
Gitleaks Scan / gitleaks (push) Successful in 5s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
ai-reviews / Review PR (pull_request) Successful in 35s
Details
PR check / Gitleaks (pull_request) Failing after 4s
Details
PR check / lint tests (pull_request) Successful in 13s
Details
PR check / handle_failures (pull_request) Successful in 2s
Details
PR check / handle_success (pull_request) Has been skipped
Details
eb5bde86d6 
This commit updates the redaction setting from `--redact=10` to `--redact=false` in the `.gitea/workflows/pr-check.yaml` file. This change ensures that no redaction occurs during the PR check process, maintaining full visibility and integrity of the data being checked.
gitea-actions bot reviewed 2026-02-14 10:15:42 +01:00
gitea-actions bot left a comment
Copy Link

Review Summary

  • .gitea/workflows/pr-check-yaml: The provided Git diff shows the deletion of a workflow file (.gitea/workflows/pr-check-yaml) containing a series of steps for CI checks. This workflow is no longer present, so it's impossible to review its logic, security, performance, and maintainability directly from the provided context. However, if we assume that this workflow was designed to perform various tests like leak testing, linting, etc., its absence could lead to potential issues in terms of maintaining proper code quality and security, as these checks are no longer being run automatically for each pull request. To ensure the integrity of new changes, it would be advisable to create a replacement workflow or revert the deletion of this file if possible. LGTM (With caution due to missing CI checks)
  • .gitea/workflows/pr-check.yaml: Summary: The provided Git diff contains a new workflow file for CI checks. It includes leak testing, linting, and handling success or failure scenarios. As there are no issues with Score >= 2 found, here's the LGTM response:
  • .gitleaks.toml: Summary: This git diff adds a configuration file for Gitleaks, a tool used for detecting secrets in code repositories. The added configuration sets up allowlists and rules to improve the tool's accuracy. LGTM
  • README.md: Summary: The provided changes in the README.md file update its title, rename a section, and modify some headings for better organization. No critical or major issues are found. However, there is a potential maintainability improvement suggestion.
# Review Summary * **.gitea/workflows/pr-check-yaml**: The provided Git diff shows the deletion of a workflow file (`.gitea/workflows/pr-check-yaml`) containing a series of steps for CI checks. This workflow is no longer present, so it's impossible to review its logic, security, performance, and maintainability directly from the provided context. However, if we assume that this workflow was designed to perform various tests like leak testing, linting, etc., its absence could lead to potential issues in terms of maintaining proper code quality and security, as these checks are no longer being run automatically for each pull request. To ensure the integrity of new changes, it would be advisable to create a replacement workflow or revert the deletion of this file if possible. **LGTM (With caution due to missing CI checks)** * **.gitea/workflows/pr-check.yaml**: **Summary:** The provided Git diff contains a new workflow file for CI checks. It includes leak testing, linting, and handling success or failure scenarios. As there are no issues with Score >= 2 found, here's the LGTM response: * **.gitleaks.toml**: **Summary:** This git diff adds a configuration file for Gitleaks, a tool used for detecting secrets in code repositories. The added configuration sets up allowlists and rules to improve the tool's accuracy. **LGTM** * **README.md**: **Summary:** The provided changes in the README.md file update its title, rename a section, and modify some headings for better organization. No critical or major issues are found. However, there is a potential maintainability improvement suggestion.
.gitea/workflows/pr-check.yaml
@@ -0,0 +13,4 @@
steps:
- name: Checkout code
uses: actions/checkout@v6
gitea-actions bot commented 2026-02-14 10:15:42 +01:00
Copy Link

[Lines 15-16] [Score: 2] Using actions/checkout v6 has been deprecated since December 2022. Consider using the v2 version, which is the latest and most actively maintained.

[Lines 15-16] [Score: 2] Using `actions/checkout` v6 has been deprecated since December 2022. Consider using the v2 version, which is the latest and most actively maintained.
.gitea/workflows/pr-check.yaml
@@ -0,0 +43,4 @@
- name: Install Node.js
uses: actions/setup-node@v4
with:
gitea-actions bot commented 2026-02-14 10:15:42 +01:00
Copy Link

[Lines 45-46] [Score: 2] Using actions/setup-node v4 has been deprecated since October 2021. Consider using the latest version, which is v3 as of now.

[Lines 45-46] [Score: 2] Using `actions/setup-node` v4 has been deprecated since October 2021. Consider using the latest version, which is v3 as of now.
.gitea/workflows/pr-check.yaml
@@ -0,0 +58,4 @@
run: |
python -m pip install --upgrade pip
pip install ansible ansible-lint yamllint
gitea-actions bot commented 2026-02-14 10:15:42 +01:00
Copy Link

[Lines 58-61] [Score: 2] Avoid using pip inside a shell script. Instead, use the actions/setup-python@v5 action to manage Python dependencies in your workflow file. This ensures that the correct environment is set up for every step.

[Lines 58-61] [Score: 2] Avoid using `pip` inside a shell script. Instead, use the `actions/setup-python@v5` action to manage Python dependencies in your workflow file. This ensures that the correct environment is set up for every step.
gitea-actions bot commented 2026-02-14 10:16:07 +01:00
Copy Link

❌ CI checks failed.

Leak: failure
Lint: success

<!--ci-failed-comment--> ❌ CI checks failed. Leak: failure Lint: success
gitea-actions bot closed this pull request 2026-02-14 10:16:07 +01:00
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 12s
Details
Gitleaks Scan / gitleaks (push) Successful in 5s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
ai-reviews / Review PR (pull_request) Successful in 35s
Details
PR check / Gitleaks (pull_request) Failing after 4s
Details
PR check / lint tests (pull_request) Successful in 13s
Details
PR check / handle_failures (pull_request) Successful in 2s
Details
PR check / handle_success (pull_request) Has been skipped
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Documentation
bug
Something is not working
 ci-failed
action test workflow failed
 ci-pass
action test workflow pass
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 source
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Jose
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Jose/ansible_role_proxmox_provision#18
Delete Branch "dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?