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 #55

Closed
wants to merge 0 commits into from

Conversation

reisene
Copy link
Owner

@reisene reisene commented Sep 24, 2024

Podsumowanie przez Sourcery

Zaktualizuj konfigurację Mergify, aby rozszerzyć harmonogram scalania dla pull requestów na głównej gałęzi.

CI:

  • Zaktualizuj konfigurację Mergify, aby zmienić harmonogram scalania z 09:00-17:00 na 09:00-03:00 w strefie czasowej Europa/Warszawa.
Original summary in English

Summary by Sourcery

Update the Mergify configuration to extend the merge schedule for pull requests on the main branch.

CI:

  • Update Mergify configuration to change the merge schedule from 09:00-17:00 to 09:00-03:00 in the Europe/Warsaw timezone.

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, aby rozszerzyć okno scalania dla głównej gałęzi. Zmiana modyfikuje warunek harmonogramu w sekcji ochrony scalania, umożliwiając scalanie od 09:00 do 03:00 (następnego dnia) w strefie czasowej Europa/Warszawa, zamiast poprzedniego okna 09:00 do 17:00.

Zmiany na poziomie plików

Zmiana Szczegóły Pliki
Rozszerzone okno scalania w konfiguracji Mergify
  • Zmieniono harmonogram scalania z '09:00-17:00' na '09:00-03:00' w strefie czasowej Europa/Warszawa
.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 to extend the merge window for the main branch. The change modifies the schedule condition in the merge protections section, allowing merges to occur from 09:00 to 03:00 (next day) in the Europe/Warsaw timezone, instead of the previous 09:00 to 17:00 window.

File-Level Changes

Change Details Files
Extended merge window in Mergify configuration
  • Changed merge schedule from '09:00-17:00' to '09:00-03:00' in the Europe/Warsaw timezone
.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.

@reisene reisene enabled auto-merge (squash) September 24, 2024 22:29
Copy link
Contributor

deepsource-io bot commented Sep 24, 2024

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

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript❌ Failure
❗ 60 occurences introduced
View Check ↗
DeepSource PHP LogoPHP✅ SuccessView Check ↗

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

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 komentarze:

  • Czy mógłbyś podać kontekst dla przedłużenia okna scalania do 03:00? Chcielibyśmy zrozumieć powody tej zmiany i upewnić się, że jest zgodna z przepływem pracy zespołu i możliwościami wsparcia.
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:

  • Could you provide some context for extending the merge window to 03:00? We'd like to understand the reasoning behind this change and ensure it aligns with the team's workflow and support capabilities.
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 reisene closed this Sep 24, 2024
auto-merge was automatically disabled September 24, 2024 22:33

Pull request was closed

@reisene reisene force-pushed the mergify/reisene/config-update branch from 444d03d to dcfe3d6 Compare September 24, 2024 22:33
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 🛑 Canceled (Inspect) Sep 24, 2024 11:19pm

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.

Do not merge outdated PRs

Make sure PRs are almost up to date before merging

  • #commits-behind <= 10

Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

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