Fix '@return null' return types

Simple cleanup

Searched for return Null and also found DrupalKernelTest but since that's a test assuming it doesn't count.

+1

Actually, I would include #5 and any other instances we can identify in the scope. Thanks!

Ops, my IDE messed with Drupal's code standards, I will try to fix it

Hiding old patch for clarity

Thanks left comment on MR.

Setting back to NR -- I think it is out of scope to remove the additional documentation about the methods being no-ops. The methods don't make any sense without that information. 🙂

Pipeline failure looks to be a random:

    PHPUnit Test failed to complete; Error: PHPUnit 9.6.8 by Sebastian Bergmann
    and contributors.
    
    Testing
    Drupal\Tests\media_library\FunctionalJavascript\FieldUiIntegrationTest
    E                                                                   1 / 1
    (100%)
    
    Time: 00:35.960, Memory: 4.00 MB
    
    There was 1 error:
    
    1)
    Drupal\Tests\media_library\FunctionalJavascript\FieldUiIntegrationTest::testFieldUiIntegration
    Behat\Mink\Exception\ExpectationException: Checkbox "set_default_value" is
    not checked, but it should be.
    

Sounds good @xjm.

Committed and pushed 8d14c6d9ad to 11.x and 15f7124382 to 10.2.x and 8b18a246b6 to 10.1.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.