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

[pre-commit.ci] pre-commit autoupdate #3270

Merged
merged 2 commits into from
Feb 5, 2024
Merged

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Dec 4, 2023

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 7a79530 to 225b1d4 Compare December 11, 2023 16:32
@botberry
Copy link
Member

botberry commented Dec 11, 2023

Hi, thanks for contributing to Strawberry 🍓!

We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!

So as soon as this PR is merged, a release will be made 🚀.

Here's an example of RELEASE.md:

Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)

Here's the tweet text:

🆕 Release (next) is out! Thanks to pre-commit-ci[bot] for the PR 👏

Get it here 👉 https://beta.strawberry.rocks/release/(next)

Copy link

codecov bot commented Dec 11, 2023

Codecov Report

Merging #3270 (40ca8c4) into main (531e37e) will increase coverage by 43.96%.
The diff coverage is 100.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3270       +/-   ##
===========================================
+ Coverage   46.98%   90.94%   +43.96%     
===========================================
  Files         493      496        +3     
  Lines       30983    31059       +76     
  Branches     3800     3806        +6     
===========================================
+ Hits        14556    28246    +13690     
+ Misses      16058     2606    -13452     
+ Partials      369      207      -162     

Copy link

codspeed-hq bot commented Dec 11, 2023

CodSpeed Performance Report

Merging #3270 will improve performances by 26.37%

Comparing pre-commit-ci-update-config (40ca8c4) with main (531e37e)

Summary

⚡ 1 improvements
✅ 12 untouched benchmarks

Benchmarks breakdown

Benchmark main pre-commit-ci-update-config Change
test_execute_basic 15.4 ms 12.2 ms +26.37%

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 1f577fa to d470a1d Compare December 25, 2023 16:31
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 203f796 to 754868b Compare January 8, 2024 16:32
@@ -14,7 +14,7 @@ repos:
exclude: (CHANGELOG|TWEET).md

- repo: https://github.com/pre-commit/mirrors-prettier
rev: v3.1.0
rev: v4.0.0-alpha.8
Copy link
Member

Choose a reason for hiding this comment

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

why does this auto-upgrade to alphas o.O

Copy link
Member

Choose a reason for hiding this comment

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

yeah, there's an issue on their repo, but it's a wontfix :'D

@@ -65,7 +65,7 @@ def __init__(self, validation_context: ValidationContext):


def count_fields_with_alias(
selection_set_owner: Union[ExecutableDefinitionNode, FieldNode, InlineFragmentNode]
selection_set_owner: Union[ExecutableDefinitionNode, FieldNode, InlineFragmentNode],
Copy link
Member

Choose a reason for hiding this comment

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

not a fan tbh :D

Copy link
Member

Choose a reason for hiding this comment

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

I don't mind

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from a1d5025 to 1d9275f Compare January 15, 2024 16:31
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from d33cb8f to 91a16fd Compare January 22, 2024 16:31
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.1.6 → v0.2.0](astral-sh/ruff-pre-commit@v0.1.6...v0.2.0)
- [github.com/pre-commit/mirrors-prettier: v3.1.0 → v4.0.0-alpha.8](pre-commit/mirrors-prettier@v3.1.0...v4.0.0-alpha.8)
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 853de44 to 364acfc Compare February 5, 2024 16:32
@patrick91 patrick91 merged commit 6123d62 into main Feb 5, 2024
60 of 62 checks passed
@patrick91 patrick91 deleted the pre-commit-ci-update-config branch February 5, 2024 16:41
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.

3 participants