pull request linting #191

Merged
paspo merged 3 commits from paspo/bglug.it:lint-pr into main 2024-08-10 15:37:29 +00:00
Owner

con questa roba facciamo dei controlli su ogni PR:

  • vengono controllati i files modificati e se sono nella directory content e hanno estensione .md vengono passati con markdownlint (c'è una config apposita e per ora ho disattivato solo il controllo della lunghezza massima della riga) e se il check fallisce, la PR non è accettabile
  • vengono controllati i files modificati e se contengono i caratteri "non permessi" che sono indicati in una lista, la PR non è accettabile

so che forse è un pò "too much", quindi ce la vediamo nei commenti

con questa roba facciamo dei controlli su ogni PR: - vengono controllati i files modificati e se sono nella directory `content` e hanno estensione `.md` vengono passati con `markdownlint` (c'è una config apposita e per ora ho disattivato solo il controllo della lunghezza massima della riga) e se il check fallisce, la PR non è accettabile - vengono controllati i files modificati e se contengono i caratteri "non permessi" che sono indicati in una lista, la PR non è accettabile so che forse è un pò "too much", quindi ce la vediamo nei commenti
paspo added 3 commits 2024-08-10 12:06:31 +00:00
fix copypaste typo
All checks were successful
Linting markdown on PR / unicode-check (pull_request) Successful in 2m20s
b6a5246ec0
paspo scheduled this pull request to auto merge when all checks succeed 2024-08-10 12:06:41 +00:00
Member

Mi sembra ottimo. Come avverrà il feedback? Un messaggio di errore all'interno di ogni PR? Che dici di fare una prova? Così capisco come funziona.

Mi sembra ottimo. Come avverrà il feedback? Un messaggio di errore all'interno di ogni PR? Che dici di fare una prova? Così capisco come funziona.
omarl approved these changes 2024-08-10 15:37:25 +00:00
paspo merged commit 4931651f29 into main 2024-08-10 15:37:29 +00:00
paspo referenced this issue from a commit 2024-08-10 15:37:30 +00:00
Author
Owner

image

e se clicchi, ti porta al workflow

![image](/attachments/5fecdb5c-4269-4677-9640-e66ec2a4f87e) e se clicchi, ti porta al workflow
paspo deleted branch lint-pr 2024-08-11 14:51:57 +00:00
Member

Visto. Figo!

Visto. Figo!
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: BgLUG/bglug.it#191
No description provided.