Incorrect behaviour for GitLab reporter on issue filtering

[alessandrozago]

This pull request is related to the issue #1127 .
Updated code and changelog.

[alessandrozago]

Hi @Ndpnt , I see that the changelog validation is failing with this message 'Error: Missing funder in the "Unreleased" section' (the validate_declarations error seems to be a one time exception).
I put the contribution acknowledgment line in the Changelog, but the error persists. Would you be able to verify what the issue is? Is it maybe just the point at the end of the line?
I did not find the correct checks it does from the other OTA projects/repositories.
Thank you

[Ndpnt]

Hi @alessandrozago,
Yes, the changelog validation error is indeed due to the missing period at the end of the acknowledgment line. Adding the period should resolve it.
Regarding the validate_declarations error, it should be fixed by rebasing your branch with the latest changes from the main branch, as the fix has already been integrated there.

[alessandrozago]

Hi, thank you for your feedback. I updated the pull request rebasing from the current main branch.
I also created a unique commit for the latest changes on the changelog, in order to have a clean history.
If there's any issue with this, I could also recreate the Pull request from scratch.

Thank you

[Ndpnt]

As this changeset corrects an unintended behavior in the software without introducing new features or breaking compatibility, it is considered as a patch in SemVer

Suggested change

	## Unreleased [minor]
	## Unreleased [patch]

[Ndpnt]

Update changelog with proper release type

[Ndpnt]

Thanks for your contribution @alessandrozago 🙏 !

[alessandrozago]

Hi @Ndpnt , sorry but I noticed that in my commits it also included a temporary directory "data (copy)" with 2 empty directories. Please let us know if we need to modify anything to clean this up, if I should made a pull request to remove it, or if you can change directly the history.

[Ndpnt]

Hi @Ndpnt , sorry but I noticed that in my commits it also included a temporary directory "data (copy)" with 2 empty directories. Please let us know if we need to modify anything to clean this up, if I should made a pull request to remove it, or if you can change directly the history.

Thank you for flagging this! It was also my responsibility to catch this during the review. That said, it would indeed be great if you could provide a pull request 👍

[alessandrozago]

Thank you for flagging this! It was also my responsibility to catch this during the review. That said, it would indeed be great if you could provide a pull request 👍

Sure. Should I just update this one or should I create a new Pull Request?