Skip to content

Conversation

@YGNT7777
Copy link
Collaborator

@YGNT7777 YGNT7777 commented Apr 23, 2025

Σχετικό Issue

closes #437

Προτεινόμενες Αλλαγές

Το GitHub Action "bottaki" ελέγχει κάθε Pull Request για αλλαγές σε «κλειδωμένα» αρχεία. Αν κάποιο από αυτά τα αρχεία περιλαμβάνεται στο lockedFiles.txt, τότε το PR μπλοκάρεται αυτόματα και δεν μπορεί να γίνει merge.

Με την προσθήκη του label "BYPASS_LABEL", ο χρήστης που υποβάλλει το Pull Request παρακάμπτει τον έλεγχο.

Ο κώδικας του είναι γραμμένος σε Python και εκτελείται μέσω Docker.
To wokflow "pr-check.yml" ελέγχει κάθε Pull Request και καλεί το action.yml το οποίο καλεί το Dockerfile

Μέχρι στιγμής για αρχεία που έχω βάλει να μην γίνονται αλλαγές είναι:
README.md
.gitignore
.gitmodules
.ruby-version
.Gemfile
LICENSE
_config.yml
index.html

Αν θέλετε να προσθέσετε καινούριο αρχείο, απλά προσθέστε το όνομα του αρχείο στο lockedFiles.txt
Για directories προσθέστε το path ξεκινώντας από το όνομα (π.χ _pages/404.html)

Demo

If there has been changes:

final

No changes

ok

Using label

withLabel

Με το που γίνει merge το pull request μου, το github action θα λειτουργεί. Δεν επηρεάζει τα κανονικά commits μόνο ελένχει στα pull requests

Υπενθυμίσεις

  • Έχω ανοίξει από πριν issue για τον καλό συντονισμό του project, το οποίο έχει πάρει το πράσινο φως με την αντίστοιχη ετικέτα
  • Έχω ενημερώσει το issueNo παραπάνω με τον αριθμό του αντίστοιχου θέματος, ώστε να κλείσει αυτόματα με την αποδοχή αυτού του αιτήματος
  • Έχω δημιουργήσει branch για τις αλλαγές

@YGNT7777 YGNT7777 changed the title Προσθήκη αυτοματισμένος έλενχος στα Pull Request Προσθήκη αυτοματισμένος έλενχος στα Pull Requests Apr 23, 2025
@netlify
Copy link

netlify bot commented Apr 23, 2025

Deploy Preview for imaginative-youtiao-564e92 ready!

Name Link
🔨 Latest commit 3b1e351
🔍 Latest deploy log https://app.netlify.com/projects/imaginative-youtiao-564e92/deploys/682710110a851c000840d5b0
😎 Deploy Preview https://deploy-preview-438--imaginative-youtiao-564e92.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.

@netlify
Copy link

netlify bot commented Apr 23, 2025

Deploy Preview for prismatic-treacle-c21935 ready!

Name Link
🔨 Latest commit 3b1e351
🔍 Latest deploy log https://app.netlify.com/projects/prismatic-treacle-c21935/deploys/68271011a9050f0008e6dc92
😎 Deploy Preview https://deploy-preview-438--prismatic-treacle-c21935.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.

@netlify
Copy link

netlify bot commented Apr 23, 2025

Deploy Preview for marvelous-alpaca-873ef8 ready!

Name Link
🔨 Latest commit 3b1e351
🔍 Latest deploy log https://app.netlify.com/projects/marvelous-alpaca-873ef8/deploys/682710111118ad000833603d
😎 Deploy Preview https://deploy-preview-438--marvelous-alpaca-873ef8.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.

@netlify
Copy link

netlify bot commented Apr 23, 2025

Deploy Preview for fantastic-gumption-d15aea ready!

Name Link
🔨 Latest commit 3b1e351
🔍 Latest deploy log https://app.netlify.com/projects/fantastic-gumption-d15aea/deploys/68271011e917740008a011f7
😎 Deploy Preview https://deploy-preview-438--fantastic-gumption-d15aea.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.

@epidrome
Copy link
Member

epidrome commented May 9, 2025

@YGNT7777 some changes and comments for improvement:

  1. Only local files no remote files.
  2. Keep consistency with platform conventions, e.g. .locked
  3. Add some documentation to the wiki, e.g., why workflow, docker, etc, provide links to external documentation

@YGNT7777
Copy link
Collaborator Author

Καλησπέρα κύριε @epidrome

Έχουν γίνει όλες οι αλλαγές, τώρα το github action διαβάζει απο το αρχείο .lockedFiles, δεν στέλνει αλλο το μήνυμα με το gif και στο wiki του ioniodi πρόσθεσα το documentation ως ξεχωριστή σελίδα, δεν το έβαλα στην αρχική σελίδα

Documentation link

Αν υπάρχει κάτι ακόμα που χρειάζεται να κάνω, πείτε μου

@stale
Copy link

stale bot commented Jul 19, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Προσθήκη αυτοματισμένος έλενχος στα Pull Request

2 participants