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.

Sign up for GitHub

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

Jump to bottom

[Translator] Mark the translator as nullable #2061

Open
wants to merge 7 commits into
base: 2.x
Choose a base branch
from
Open

[Translator] Mark the translator as nullable #2061

wants to merge 7 commits into from

Commits on Aug 14, 2024

  1. Make the translator null when its invalid.

    Sander Verkuil authored and Sander Verkuil committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    5b68aa3 View commit details
    Browse the repository at this point in the history

  2. Updated documentation

    Sander Verkuil authored and Sander Verkuil committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    d06db23 View commit details
    Browse the repository at this point in the history

  3. Added logger and added a test scenario

    Sander Verkuil authored and Sander Verkuil committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    60ca5b6 View commit details
    Browse the repository at this point in the history

  4. chore: php-cs-fixer

    Sander Verkuil authored and Sander Verkuil committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    45ade05 View commit details
    Browse the repository at this point in the history

  5. Update src/Translator/doc/index.rst 

    Co-authored-by: Alexis Lefebvre <[email protected]>
    @SanderVerkuil @alexislefebvre
    2 people authored and Sander Verkuil committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    8219a2d View commit details
    Browse the repository at this point in the history

  6. The TranslatorBag is no TranslatorInterface 

    Leverage union types to properly set up the configuration
    @SanderVerkuil
    SanderVerkuil authored and Sander Verkuil committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    a548681 View commit details
    Browse the repository at this point in the history

  7. Re-added the test for the container with disabled translator

    @SanderVerkuil
    SanderVerkuil committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    6143bbe View commit details
    Browse the repository at this point in the history