Add International Womens Day to DE-MV

[ihmels]

No description provided.

[stelgenhof]

Can you add the appropriate unit tests for your change? Also, if you have any documentation or reference for this change, that would be great.

[stelgenhof]

@ihmels Can you add the appropriate unit tests for your change? Also, if you have any documentation or reference for this change, that would be great.

[ihmels]

I have added a test.

The source is https://www.ndr.de/nachrichten/mecklenburg-vorpommern/Frauentag-in-MV-Landtag-beschliesst-neuen-Feiertag,frauentag370.html. However, there is another source on Wikipedia that is already referenced via Germany::getSources().

Where is the best place to put the source?

[stelgenhof]

@ihmels Good point regarding the source. You could override the getSources method while adding the additional link.

[ihmels]

@stelgenhof I've added the source and fixed the namespace in the test. Can you run the workflow again for these changes?

[ihmels]

@stelgenhof Can you approve the workflow again?

[stelgenhof]

@ihmels Can you check you once more? There are still some errors.

[ihmels]

@ihmels Can you check you once more?

@stelgenhof The error was:

ERROR: InvalidReturnStatement - src/Yasumi/Yasumi.php:245:16 - The inferred type 'array<string, array<array-key, string>|null|string>' does not match the declared return type 'array<string, string>' for Yasumi\Yasumi::getProviders (see https://psalm.dev/128)
       return $providers

This error was not related to my changes, but I fixed it. Can you run tests again?

[stelgenhof]

@ihmels I've made some fixes on the develop branch that would make your PR error free. Can you rebase your branch? Thanks.