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

Update label.yml #131

Merged
merged 2 commits into from
Oct 5, 2024
Merged

Update label.yml #131

merged 2 commits into from
Oct 5, 2024

Conversation

reisene
Copy link
Owner

@reisene reisene commented Oct 5, 2024

reisene TypeError: Cannot read properties of null (reading 'includes') Powered by Pull Request Badge

Podsumowanie przez Sourcery

Popraw przepływ pracy GitHub Actions, aby prawidłowo dodawać etykiety do pull requestów, przechodząc z API issues do API pulls.

CI:

  • Napraw przepływ pracy GitHub Actions, aby poprawnie dodawać etykiety do pull requestów, używając metody 'github.pulls.addLabel' zamiast 'github.issues.addLabel'.
Original summary in English

Summary by Sourcery

Correct the GitHub Actions workflow to properly add labels to pull requests by switching from the issues API to the pulls API.

CI:

  • Fix the GitHub Actions workflow to correctly add labels to pull requests by using the 'github.pulls.addLabel' method instead of 'github.issues.addLabel'.

Copy link
Contributor

aviator-app bot commented Oct 5, 2024

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged using Aviator.


See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

Copy link
Contributor

aviator-app bot commented Oct 5, 2024

Aviator Changeset actions:

Copy link

Review changes with SemanticDiff.

Copy link

vercel bot commented Oct 5, 2024

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

Name Status Preview Comments Updated (UTC)
hulaj-dusza-serwis ❌ Failed (Inspect) Oct 5, 2024 5:29am

Copy link

Please follow naming conventions! 😿

Copy link

github-actions bot commented Oct 5, 2024

Click here to review and test in web IDE: Contribute

Copy link

github-actions bot commented Oct 5, 2024

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

.github/workflows/label.yml
  • actions/github-script@5.*.*

Copy link
Contributor

sourcery-ai bot commented Oct 5, 2024

Przewodnik Recenzenta od Sourcery

Ten pull request aktualizuje plik workflow GitHub Actions 'label.yml', aby zmodyfikować skrypt, który dodaje etykietę 'bot' do pull requestów. Zmiana zastępuje użycie 'github.issues.addLabel' na 'github.pulls.addLabel' i zawiera dodatkowe parametry dla właściciela repozytorium i nazwy.

Nie wygenerowano diagramów, ponieważ zmiany wyglądają na proste i nie wymagają wizualnej reprezentacji.

Zmiany na poziomie pliku

Zmiana Szczegóły Pliki
Zastąpienie 'github.issues.addLabel' na 'github.pulls.addLabel' w celu dodawania etykiet do pull requestów
  • Usuń wywołanie funkcji 'github.issues.addLabel'
  • Dodaj wywołanie funkcji 'github.pulls.addLabel'
  • Uwzględnij parametry 'owner' i 'repo' w nowym wywołaniu funkcji
  • Zmień nazwę parametru 'issue_number' na 'pull_number'
.github/workflows/label.yml

Wskazówki i polecenia

Interakcja z Sourcery

  • Wywołaj nową recenzję: Skomentuj @sourcery-ai review w pull request.
  • Kontynuuj dyskusje: Odpowiedz bezpośrednio na komentarze recenzji Sourcery.
  • Wygeneruj problem GitHub z komentarza recenzji: Poproś Sourcery o utworzenie
    problemu z komentarza recenzji, odpowiadając na niego.
  • Wygeneruj tytuł pull request: Napisz @sourcery-ai gdziekolwiek w tytule pull
    request, aby wygenerować tytuł w dowolnym momencie.
  • Wygeneruj podsumowanie pull request: Napisz @sourcery-ai summary gdziekolwiek w
    treści pull request, aby wygenerować podsumowanie PR w dowolnym momencie. Możesz również użyć
    tego polecenia, aby określić, gdzie powinno zostać wstawione podsumowanie.

Dostosowywanie swojego doświadczenia

Uzyskaj dostęp do swojego pulpitu, aby:

  • Włączyć lub wyłączyć funkcje recenzji, takie jak generowane przez Sourcery podsumowanie pull request,
    przewodnik recenzenta i inne.
  • Zmienić język recenzji.
  • Dodawać, usuwać lub edytować niestandardowe instrukcje recenzji.
  • Dostosować inne ustawienia recenzji.

Uzyskiwanie pomocy

Original review guide in English

Reviewer's Guide by Sourcery

This pull request updates the GitHub Actions workflow file 'label.yml' to modify the script that adds a 'bot' label to pull requests. The change replaces the use of 'github.issues.addLabel' with 'github.pulls.addLabel' and includes additional parameters for the repository owner and name.

No diagrams generated as the changes look simple and do not need a visual representation.

File-Level Changes

Change Details Files
Replace 'github.issues.addLabel' with 'github.pulls.addLabel' for adding labels to pull requests
  • Remove 'github.issues.addLabel' function call
  • Add 'github.pulls.addLabel' function call
  • Include 'owner' and 'repo' parameters in the new function call
  • Rename 'issue_number' parameter to 'pull_number'
.github/workflows/label.yml

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time. You can also use
    this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link
Contributor

deepsource-io bot commented Oct 5, 2024

Here's the code health analysis summary for commits 54e299f..22cfc1c. 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 Oct 5, 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

Copy link
Contributor

aviator-app bot commented Oct 5, 2024

This pull request failed to merge: new commit introduced for a queued PR, invalidating the status. After you have resolved the problem, you should remove the blocked pull request label from this PR and then try to re-queue the PR. Note that the pull request will be automatically re-queued if it has the mergequeue label.

Associated Draft PR #132

@reisene reisene self-assigned this Oct 5, 2024
@reisene reisene removed the blocked label Oct 5, 2024
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, a ja wykorzystam opinie, aby ulepszyć twoje recenzje.
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 and I'll use the feedback to improve your reviews.

Copy link

sonarqubecloud bot commented Oct 5, 2024

@reisene
Copy link
Owner Author

reisene commented Oct 5, 2024

/aviator merge

Copy link
Contributor

aviator-app bot commented Oct 5, 2024

Aviator has accepted the merge request. It will enter the queue when all of the required status checks have passed. Aviator will update the sticky status comment as the pull request moves through the queue.

Copy link
Contributor

aviator-app bot commented Oct 5, 2024

Skipping bot pull request creation because the queue is empty and this pull request is up to date with main.

@aviator-app aviator-app bot merged commit 79f6a58 into main Oct 5, 2024
29 of 37 checks passed
@aviator-app aviator-app bot deleted the reisene-patch-1 branch October 5, 2024 05:11
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.

1 participant