Skip to content

refactor: improve workflow maintainability and consistency#6

Open
MatrixNeoKozak wants to merge 1 commit into
echs-top:mainfrom
MatrixNeoKozak:fix/improvement-1780781255744
Open

refactor: improve workflow maintainability and consistency#6
MatrixNeoKozak wants to merge 1 commit into
echs-top:mainfrom
MatrixNeoKozak:fix/improvement-1780781255744

Conversation

@MatrixNeoKozak

Copy link
Copy Markdown

This PR refactors the GitHub Actions workflows to improve maintainability and consistency.

Summary of Changes

  1. Extracted the monolithic Python script from update-add-rules.yml into a dedicated file scripts/fetch_rules.py.
  2. Standardized Git operations across update-add-rules.yml and update-list-rules.yml by replacing manual git commands with stefanzweifel/git-auto-commit-action@v5.

Rationale

  • Maintainability: Moving the Python logic out of the YAML file makes the code easier to read, test, and maintain.
  • Consistency & Reliability: Using a centralized, well-maintained action for Git commits (stefanzweifel/git-auto-commit-action) reduces boilerplate code and improves the reliability of the Git operations compared to manual configurations within each workflow.

Technical Impact

  • Workflows remain functionally equivalent but with improved modularity and structural integrity.
  • Reduced risk of errors in manual Git configuration.

@vercel

vercel Bot commented Jun 6, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the ECHSFXY's projects Team on Vercel.

A member of the Team first needs to authorize it.

@netlify

netlify Bot commented Jun 6, 2026

Copy link
Copy Markdown

Deploy Preview for mihomocdn ready!

Name Link
🔨 Latest commit 867af64
🔍 Latest deploy log https://app.netlify.com/projects/mihomocdn/deploys/6a2490cc78a6570007372169
😎 Deploy Preview https://deploy-preview-6--mihomocdn.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant