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

Release (#598) #599

Merged
merged 1 commit into from
Nov 9, 2024
Merged

Release (#598) #599

merged 1 commit into from
Nov 9, 2024

Conversation

philipbrembeck
Copy link
Contributor

@philipbrembeck philipbrembeck commented Nov 9, 2024

  • feat: Init v1 ingredients

  • feat(ESLint): Use flatConfig (feat(ESLint): Use flatConfig #574)

  • chore: Improve lists

  • chore(ESLint): Use flatConfig

  • build(deps-dev): Bump @typescript-eslint/eslint-plugin

Bumps @typescript-eslint/eslint-plugin from 8.11.0 to 8.12.1.


updated-dependencies:

  • dependency-name: "@typescript-eslint/eslint-plugin" dependency-type: direct:development update-type: version-update:semver-minor ...
  • build(deps-dev): Bump @nestjs/schematics from 10.2.2 to 10.2.3

Bumps @nestjs/schematics from 10.2.2 to 10.2.3.


updated-dependencies:

  • dependency-name: "@nestjs/schematics" dependency-type: direct:development update-type: version-update:semver-patch ...
  • build(deps-dev): Bump @typescript-eslint/parser from 8.11.0 to 8.12.1

Bumps @typescript-eslint/parser from 8.11.0 to 8.12.1.


updated-dependencies:

  • dependency-name: "@typescript-eslint/parser" dependency-type: direct:development update-type: version-update:semver-minor ...
  • build(deps-dev): Bump @typescript-eslint/parser from 8.12.1 to 8.12.2

Bumps @typescript-eslint/parser from 8.12.1 to 8.12.2.


updated-dependencies:

  • dependency-name: "@typescript-eslint/parser" dependency-type: direct:development update-type: version-update:semver-patch ...
  • build(deps): Bump @nestjs/axios from 3.1.0 to 3.1.1

Bumps @nestjs/axios from 3.1.0 to 3.1.1.


updated-dependencies:

  • dependency-name: "@nestjs/axios" dependency-type: direct:production update-type: version-update:semver-patch ...
  • build(deps-dev): Bump @types/lodash from 4.17.12 to 4.17.13

Bumps @types/lodash from 4.17.12 to 4.17.13.


updated-dependencies:

  • dependency-name: "@types/lodash" dependency-type: direct:development update-type: version-update:semver-patch ...
  • build(deps-dev): Bump @typescript-eslint/eslint-plugin

Bumps @typescript-eslint/eslint-plugin from 8.12.1 to 8.12.2.


updated-dependencies:

  • dependency-name: "@typescript-eslint/eslint-plugin" dependency-type: direct:development update-type: version-update:semver-patch ...
  • build(deps-dev): Bump eslint-plugin-n from 17.11.1 to 17.12.0

Bumps eslint-plugin-n from 17.11.1 to 17.12.0.


updated-dependencies:

  • dependency-name: eslint-plugin-n dependency-type: direct:development update-type: version-update:semver-minor ...
  • build(deps-dev): Bump eslint from 9.13.0 to 9.14.0

Bumps eslint from 9.13.0 to 9.14.0.


updated-dependencies:

  • dependency-name: eslint dependency-type: direct:development update-type: version-update:semver-minor ...
  • build(deps-dev): Bump @nestjs/cli from 10.4.5 to 10.4.7

Bumps @nestjs/cli from 10.4.5 to 10.4.7.


updated-dependencies:

  • dependency-name: "@nestjs/cli" dependency-type: direct:development update-type: version-update:semver-patch ...
  • build(deps-dev): Bump @typescript-eslint/eslint-plugin

Bumps @typescript-eslint/eslint-plugin from 8.12.2 to 8.13.0.


updated-dependencies:

  • dependency-name: "@typescript-eslint/eslint-plugin" dependency-type: direct:development update-type: version-update:semver-minor ...
  • build(deps): Bump @nestjs/common from 10.4.6 to 10.4.7

Bumps @nestjs/common from 10.4.6 to 10.4.7.


updated-dependencies:

  • dependency-name: "@nestjs/common" dependency-type: direct:production update-type: version-update:semver-patch ...
  • build(deps-dev): Bump @typescript-eslint/parser from 8.12.2 to 8.13.0

Bumps @typescript-eslint/parser from 8.12.2 to 8.13.0.


updated-dependencies:

  • dependency-name: "@typescript-eslint/parser" dependency-type: direct:development update-type: version-update:semver-minor ...
  • build(deps-dev): Bump @nestjs/testing from 10.4.6 to 10.4.7

Bumps @nestjs/testing from 10.4.6 to 10.4.7.


updated-dependencies:

  • dependency-name: "@nestjs/testing" dependency-type: direct:development update-type: version-update:semver-patch ...
  • build(deps): Bump @nestjs/platform-express from 10.4.6 to 10.4.7

Bumps @nestjs/platform-express from 10.4.6 to 10.4.7.


updated-dependencies:

  • dependency-name: "@nestjs/platform-express" dependency-type: direct:production update-type: version-update:semver-patch ...
  • build(deps): Bump @nestjs/core from 10.4.6 to 10.4.7

Bumps @nestjs/core from 10.4.6 to 10.4.7.


updated-dependencies:

  • dependency-name: "@nestjs/core" dependency-type: direct:production update-type: version-update:semver-patch ...
  • build(deps-dev): Bump eslint-plugin-n from 17.12.0 to 17.13.0

Bumps eslint-plugin-n from 17.12.0 to 17.13.0.


updated-dependencies:

  • dependency-name: eslint-plugin-n dependency-type: direct:development update-type: version-update:semver-minor ...
  • build(deps): Bump @nestjs/axios from 3.1.1 to 3.1.2

Bumps @nestjs/axios from 3.1.1 to 3.1.2.


updated-dependencies:

  • dependency-name: "@nestjs/axios" dependency-type: direct:production update-type: version-update:semver-patch ...
  • build(deps-dev): Bump eslint-plugin-n from 17.13.0 to 17.13.1

Bumps eslint-plugin-n from 17.13.0 to 17.13.1.


updated-dependencies:

  • dependency-name: eslint-plugin-n dependency-type: direct:development update-type: version-update:semver-patch ...

Bumps @nestjs/swagger from 7.4.2 to 8.0.3.


updated-dependencies:

  • dependency-name: "@nestjs/swagger" dependency-type: direct:production update-type: version-update:semver-major ...

Summary by Sourcery

Release new version with v1 ingredients feature, update ESLint configuration to use flatConfig, and upgrade multiple dependencies to their latest versions for improved stability and performance.

New Features:

  • Introduce v1 ingredients feature.

Enhancements:

  • Switch to using flatConfig for ESLint configuration.

Build:

  • Update various dependencies including @nestjs/axios, @nestjs/common, @nestjs/core, @nestjs/platform-express, @nestjs/swagger, and others to their latest versions.

* feat: Init v1 ingredients

* feat(ESLint): Use flatConfig (#574)

* chore: Improve lists

* chore(ESLint): Use flatConfig

* build(deps-dev): Bump @typescript-eslint/eslint-plugin

Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 8.11.0 to 8.12.1.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.12.1/packages/eslint-plugin)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump @nestjs/schematics from 10.2.2 to 10.2.3

Bumps [@nestjs/schematics](https://github.com/nestjs/schematics) from 10.2.2 to 10.2.3.
- [Release notes](https://github.com/nestjs/schematics/releases)
- [Changelog](https://github.com/nestjs/schematics/blob/master/.release-it.json)
- [Commits](nestjs/schematics@10.2.2...10.2.3)

---
updated-dependencies:
- dependency-name: "@nestjs/schematics"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump @typescript-eslint/parser from 8.11.0 to 8.12.1

Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 8.11.0 to 8.12.1.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.12.1/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump @typescript-eslint/parser from 8.12.1 to 8.12.2

Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 8.12.1 to 8.12.2.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.12.2/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps): Bump @nestjs/axios from 3.1.0 to 3.1.1

Bumps [@nestjs/axios](https://github.com/nestjs/axios) from 3.1.0 to 3.1.1.
- [Release notes](https://github.com/nestjs/axios/releases)
- [Changelog](https://github.com/nestjs/axios/blob/master/.release-it.json)
- [Commits](nestjs/axios@3.1.0...3.1.1)

---
updated-dependencies:
- dependency-name: "@nestjs/axios"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump @types/lodash from 4.17.12 to 4.17.13

Bumps [@types/lodash](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/lodash) from 4.17.12 to 4.17.13.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/lodash)

---
updated-dependencies:
- dependency-name: "@types/lodash"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump @typescript-eslint/eslint-plugin

Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 8.12.1 to 8.12.2.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.12.2/packages/eslint-plugin)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump eslint-plugin-n from 17.11.1 to 17.12.0

Bumps [eslint-plugin-n](https://github.com/eslint-community/eslint-plugin-n) from 17.11.1 to 17.12.0.
- [Release notes](https://github.com/eslint-community/eslint-plugin-n/releases)
- [Changelog](https://github.com/eslint-community/eslint-plugin-n/blob/master/CHANGELOG.md)
- [Commits](eslint-community/eslint-plugin-n@v17.11.1...v17.12.0)

---
updated-dependencies:
- dependency-name: eslint-plugin-n
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump eslint from 9.13.0 to 9.14.0

Bumps [eslint](https://github.com/eslint/eslint) from 9.13.0 to 9.14.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](eslint/eslint@v9.13.0...v9.14.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump @nestjs/cli from 10.4.5 to 10.4.7

Bumps [@nestjs/cli](https://github.com/nestjs/nest-cli) from 10.4.5 to 10.4.7.
- [Release notes](https://github.com/nestjs/nest-cli/releases)
- [Changelog](https://github.com/nestjs/nest-cli/blob/master/.release-it.json)
- [Commits](nestjs/nest-cli@10.4.5...10.4.7)

---
updated-dependencies:
- dependency-name: "@nestjs/cli"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump @typescript-eslint/eslint-plugin

Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 8.12.2 to 8.13.0.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.13.0/packages/eslint-plugin)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps): Bump @nestjs/common from 10.4.6 to 10.4.7

Bumps [@nestjs/common](https://github.com/nestjs/nest/tree/HEAD/packages/common) from 10.4.6 to 10.4.7.
- [Release notes](https://github.com/nestjs/nest/releases)
- [Commits](https://github.com/nestjs/nest/commits/v10.4.7/packages/common)

---
updated-dependencies:
- dependency-name: "@nestjs/common"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump @typescript-eslint/parser from 8.12.2 to 8.13.0

Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 8.12.2 to 8.13.0.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.13.0/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump @nestjs/testing from 10.4.6 to 10.4.7

Bumps [@nestjs/testing](https://github.com/nestjs/nest/tree/HEAD/packages/testing) from 10.4.6 to 10.4.7.
- [Release notes](https://github.com/nestjs/nest/releases)
- [Commits](https://github.com/nestjs/nest/commits/v10.4.7/packages/testing)

---
updated-dependencies:
- dependency-name: "@nestjs/testing"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps): Bump @nestjs/platform-express from 10.4.6 to 10.4.7

Bumps [@nestjs/platform-express](https://github.com/nestjs/nest/tree/HEAD/packages/platform-express) from 10.4.6 to 10.4.7.
- [Release notes](https://github.com/nestjs/nest/releases)
- [Commits](https://github.com/nestjs/nest/commits/v10.4.7/packages/platform-express)

---
updated-dependencies:
- dependency-name: "@nestjs/platform-express"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps): Bump @nestjs/core from 10.4.6 to 10.4.7

Bumps [@nestjs/core](https://github.com/nestjs/nest/tree/HEAD/packages/core) from 10.4.6 to 10.4.7.
- [Release notes](https://github.com/nestjs/nest/releases)
- [Commits](https://github.com/nestjs/nest/commits/v10.4.7/packages/core)

---
updated-dependencies:
- dependency-name: "@nestjs/core"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump eslint-plugin-n from 17.12.0 to 17.13.0

Bumps [eslint-plugin-n](https://github.com/eslint-community/eslint-plugin-n) from 17.12.0 to 17.13.0.
- [Release notes](https://github.com/eslint-community/eslint-plugin-n/releases)
- [Changelog](https://github.com/eslint-community/eslint-plugin-n/blob/master/CHANGELOG.md)
- [Commits](eslint-community/eslint-plugin-n@v17.12.0...v17.13.0)

---
updated-dependencies:
- dependency-name: eslint-plugin-n
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps): Bump @nestjs/axios from 3.1.1 to 3.1.2

Bumps [@nestjs/axios](https://github.com/nestjs/axios) from 3.1.1 to 3.1.2.
- [Release notes](https://github.com/nestjs/axios/releases)
- [Changelog](https://github.com/nestjs/axios/blob/master/.release-it.json)
- [Commits](nestjs/axios@3.1.1...3.1.2)

---
updated-dependencies:
- dependency-name: "@nestjs/axios"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps-dev): Bump eslint-plugin-n from 17.13.0 to 17.13.1

Bumps [eslint-plugin-n](https://github.com/eslint-community/eslint-plugin-n) from 17.13.0 to 17.13.1.
- [Release notes](https://github.com/eslint-community/eslint-plugin-n/releases)
- [Changelog](https://github.com/eslint-community/eslint-plugin-n/blob/master/CHANGELOG.md)
- [Commits](eslint-community/eslint-plugin-n@v17.13.0...v17.13.1)

---
updated-dependencies:
- dependency-name: eslint-plugin-n
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps): Bump @nestjs/swagger from 7.4.2 to 8.0.3 (#597)

Bumps [@nestjs/swagger](https://github.com/nestjs/swagger) from 7.4.2 to 8.0.3.
- [Release notes](https://github.com/nestjs/swagger/releases)
- [Changelog](https://github.com/nestjs/swagger/blob/master/.release-it.json)
- [Commits](nestjs/swagger@7.4.2...8.0.3)

---
updated-dependencies:
- dependency-name: "@nestjs/swagger"
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Copy link

stackblitz bot commented Nov 9, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link

sourcery-ai bot commented Nov 9, 2024

Reviewer's Guide by Sourcery

This pull request includes several dependency updates, ESLint configuration changes, and code organization improvements. The main changes involve upgrading NestJS packages, migrating to ESLint flat config, and reorganizing imports across multiple files.

Class diagram for updated imports in GradesController

classDiagram
    class GradesController {
        - Logger logger
    }
    class GradesService
    class Response
    class ApiResponse
    class ApiTags
    class ApiBody
    class BarcodeDto
    class backendResponseDto

    GradesController --> GradesService
    GradesController --> Response
    GradesController --> ApiResponse
    GradesController --> ApiTags
    GradesController --> ApiBody
    GradesController --> BarcodeDto
    GradesController --> backendResponseDto
Loading

Class diagram for updated imports in ProductController

classDiagram
    class ProductController {
        - Logger logger
    }
    class ProductService
    class Response
    class ApiResponse
    class ApiTags

    ProductController --> ProductService
    ProductController --> Response
    ProductController --> ApiResponse
    ProductController --> ApiTags
Loading

Class diagram for updated imports in PetaController

classDiagram
    class petaType {
        + ApiProperty()
    }
    class Response
    class ApiResponse
    class ApiTags
    class ApiExcludeEndpoint

    petaType --> Response
    petaType --> ApiResponse
    petaType --> ApiTags
    petaType --> ApiExcludeEndpoint
Loading

Class diagram for updated imports in GradesService

classDiagram
    class GradesService {
        - Logger logger
    }
    class HttpService
    class AxiosResponse
    class Observable

    GradesService --> HttpService
    GradesService --> AxiosResponse
    GradesService --> Observable
Loading

File-Level Changes

Change Details Files
Upgraded multiple NestJS framework packages
  • Updated @nestjs/axios from 3.1.0 to 3.1.2
  • Updated @nestjs/common from 10.4.6 to 10.4.7
  • Updated @nestjs/core from 10.4.6 to 10.4.7
  • Updated @nestjs/swagger from 7.4.2 to 8.0.3
  • Updated @nestjs/cli from 10.4.5 to 10.4.7
package.json
pnpm-lock.yaml
Migrated to ESLint flat configuration system
  • Removed .eslintrc.js and .eslintignore in favor of eslint.config.mjs
  • Added new ESLint flat config file with TypeScript and import rules
  • Updated ESLint dependencies to latest versions
  • Added import sorting rules configuration
.eslintrc.js
.eslintignore
eslint.config.mjs
package.json
Reorganized imports across multiple source files
  • Grouped imports by type (built-in, external, internal)
  • Added newlines between import groups
  • Alphabetically sorted imports within groups
src/grades/grades.controller.ts
src/main.ts
src/product/product.controller.ts
src/peta/peta.controller.ts
src/grades/grades.service.ts
src/product/tests/product.controller.spec.ts
Updated ingredient classifications
  • Moved 'lecithin' from ismaybenotvegan.json to isvegan.json
ismaybenotvegan.json
isvegan.json

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

@philipbrembeck philipbrembeck merged commit 96937b3 into staging Nov 9, 2024
9 checks passed
Copy link

@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.

Hey @philipbrembeck - 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.

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