Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(Mergify): configuration update #47

Merged
merged 1 commit into from
Sep 24, 2024
Merged

Conversation

reisene
Copy link
Owner

@reisene reisene commented Sep 24, 2024

This change has been made by @reisene from the Mergify merge protections editor.

Podsumowanie przez Sourcery

Zaktualizuj konfigurację Mergify, aby dodać zabezpieczenia scalania, w tym ograniczenia dotyczące scalania w godzinach pracy oraz upewnienie się, że PR-y nie są nieaktualne.

CI:

  • Zaktualizuj konfigurację Mergify, aby uwzględnić zabezpieczenia scalania, które ograniczają scalanie do godzin pracy i zapewniają, że PR-y są aktualne.
Original summary in English

Summary by Sourcery

Update Mergify configuration to add merge protections, including restrictions on merging during working hours and ensuring PRs are not outdated.

CI:

  • Update Mergify configuration to include merge protections that restrict merging to working hours and ensure PRs are up to date.

Copy link

vercel bot commented Sep 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
hulaj-dusza-serwis ❌ Failed (Inspect) Sep 24, 2024 9:12pm

Copy link

Review changes with SemanticDiff.

Copy link
Contributor

sourcery-ai bot commented Sep 24, 2024

Przewodnik Recenzenta od Sourcery

Ten pull request aktualizuje plik konfiguracyjny Mergify (.mergify.yml), aby dodać nowe zabezpieczenia scalania. Zmiany wprowadzają dwie nowe zasady: jedną, aby ograniczyć scalanie do godzin pracy, oraz drugą, aby zapobiec scalaniu nieaktualnych pull requestów.

Zmiany na poziomie pliku

Zmiana Szczegóły Pliki
Dodano nowe zabezpieczenia scalania do konfiguracji Mergify
  • Wprowadzono zabezpieczenie 'Scalaj w godzinach pracy'
  • Ustawiono godziny pracy na poniedziałek-piątek, 09:00-17:00 w strefie czasowej Polska/Warszawa
  • Dodano zabezpieczenie 'Nie scalaj nieaktualnych PR'
  • Ustawiono próg nie więcej niż 10 commitów wstecz, aby PR były uznawane za aktualne
.mergify.yml

Diagram sekwencji

Nie wygenerowano diagramu sekwencji.


Wskazówki
  • Wywołaj nową recenzję Sourcery, komentując @sourcery-ai review w pull request.
  • Kontynuuj dyskusję z Sourcery, odpowiadając bezpośrednio na komentarze recenzji.
  • Możesz zmienić ustawienia recenzji w dowolnym momencie, uzyskując dostęp do swojego pulpitu:
    • Włącz lub wyłącz podsumowanie pull request generowane przez Sourcery lub przewodnik recenzenta;
    • Zmień język recenzji;
  • Zawsze możesz skontaktować się z nami, jeśli masz jakiekolwiek pytania lub uwagi.
Original review guide in English

Reviewer's Guide by Sourcery

This pull request updates the Mergify configuration file (.mergify.yml) to add new merge protections. The changes introduce two new rules: one to restrict merging to working hours, and another to prevent merging of outdated pull requests.

File-Level Changes

Change Details Files
Added new merge protections to the Mergify configuration
  • Introduced a 'Merge during working hours' protection
  • Set working hours to Monday-Friday, 09:00-17:00 in the Poland/Warsaw timezone
  • Added a 'Do not merge outdated PRs' protection
  • Set a threshold of no more than 10 commits behind for PRs to be considered up-to-date
.mergify.yml

Sequence Diagram

No sequence diagram generated.


Tips
  • Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
  • Continue your discussion with Sourcery by replying directly to review comments.
  • You can change your review settings at any time by accessing your dashboard:
    • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
    • Change the review language;
  • You can always contact us if you have any questions or feedback.

Copy link
Contributor

deepsource-io bot commented Sep 24, 2024

Here's the code health analysis summary for commits 0bc340d..f8512e8. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗
DeepSource PHP LogoPHP✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

Copy link
Contributor

mergify bot commented Sep 24, 2024

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

📃 Configuration Change Requirements

Mergify configuration change

  • check-success = Configuration changed

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hej @reisene - Przejrzałem twoje zmiany - oto kilka uwag:

Ogólne uwagi:

  • Rozważ wpływ ograniczenia czasowego na globalne zespoły. Może być korzystne, aby uczynić to bardziej elastycznym lub usunąć, jeśli twój projekt ma współpracowników z różnych stref czasowych.
  • Przeanalizuj, czy 10 commitów wstecz to odpowiedni próg dla tempa twojego projektu. W zależności od tego, jak szybko porusza się twoja główna gałąź, może to wymagać dostosowania.
Oto, co sprawdziłem podczas przeglądu
  • 🟢 Ogólne kwestie: wszystko wygląda dobrze
  • 🟢 Bezpieczeństwo: wszystko wygląda dobrze
  • 🟢 Testowanie: wszystko wygląda dobrze
  • 🟢 Złożoność: wszystko wygląda dobrze
  • 🟢 Dokumentacja: wszystko wygląda dobrze

Sourcery jest darmowe dla open source - jeśli podobają ci się nasze recenzje, rozważ ich udostępnienie ✨
Pomóż mi być bardziej użytecznym! Proszę kliknij 👍 lub 👎 przy każdym komentarzu, aby powiedzieć mi, czy był pomocny.
Original comment in English

Hey @reisene - I've reviewed your changes - here's some feedback:

Overall Comments:

  • Consider the impact of the time restriction on global teams. It might be beneficial to make this more flexible or remove it if your project has contributors across different time zones.
  • Review if 10 commits behind is an appropriate threshold for your project's pace. Depending on how quickly your main branch moves, this might need to be adjusted.
Here's what I looked at during the review
  • 🟢 General issues: all looks good
  • 🟢 Security: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.

@reisene
Copy link
Owner Author

reisene commented Sep 24, 2024

@mergify queue

Copy link
Contributor

mergify bot commented Sep 24, 2024

queue

🟠 Waiting for conditions to match

  • any of: [📌 queue -> configuration change requirements]
    • -mergify-configuration-changed
    • check-success = Configuration changed
  • any of: [🔀 queue conditions]
    • all of: [📌 queue conditions of queue default]
      • any of: [🛡 GitHub branch protection]
        • check-neutral = Mergify Merge Protections
        • check-skipped = Mergify Merge Protections
        • check-success = Mergify Merge Protections
  • -closed [📌 queue requirement]
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]

@reisene
Copy link
Owner Author

reisene commented Sep 24, 2024

@mergify queue

Copy link
Contributor

mergify bot commented Sep 24, 2024

queue

🟠 Waiting for conditions to match

  • any of: [📌 queue -> configuration change requirements]
    • -mergify-configuration-changed
    • check-success = Configuration changed
  • any of: [🔀 queue conditions]
    • all of: [📌 queue conditions of queue default]
      • any of: [🛡 GitHub branch protection]
        • check-neutral = Mergify Merge Protections
        • check-skipped = Mergify Merge Protections
        • check-success = Mergify Merge Protections
  • -closed [📌 queue requirement]
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]

@reisene reisene merged commit 756fdee into main Sep 24, 2024
12 of 20 checks passed
@reisene reisene deleted the mergify/reisene/config-update branch September 24, 2024 20:57
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