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

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 używać strefy czasowej 'Europe/Warsaw' dla ochrony scalania, zapewniając zgodność ze standardowymi konwencjami nazewnictwa stref czasowych.

CI:

  • Zaktualizuj konfigurację Mergify, aby zmienić strefę czasową z 'Poland/Warsaw' na 'Europe/Warsaw' dla ochrony scalania.
Original summary in English

Summary by Sourcery

Update the Mergify configuration to use the 'Europe/Warsaw' timezone for merge protections, ensuring consistency with standard timezone naming conventions.

CI:

  • Update the Mergify configuration to change the timezone from 'Poland/Warsaw' to 'Europe/Warsaw' for merge protections.

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 10:33pm

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 poprawić specyfikację strefy czasowej dla harmonogramu scalania. Zmiana zastępuje 'Poland/Warsaw' na 'Europe/Warsaw' w warunku harmonogramu.

Zmiany na poziomie pliku

Zmiana Szczegóły Pliki
Aktualizacja specyfikacji strefy czasowej w konfiguracji Mergify
  • Zmień strefę czasową z 'Poland/Warsaw' na 'Europe/Warsaw'
.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 correct the time zone specification for the merge schedule. The change replaces 'Poland/Warsaw' with 'Europe/Warsaw' in the schedule condition.

File-Level Changes

Change Details Files
Update time zone specification in Mergify configuration
  • Change time zone from 'Poland/Warsaw' to 'Europe/Warsaw'
.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 9682a98..7b23139. 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

@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 i wyglądają świetnie!

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 and they look great!

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 merged commit 277a62c into main Sep 24, 2024
14 of 19 checks passed
@reisene reisene deleted the mergify/reisene/config-update branch September 24, 2024 22:14
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

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