[Feature Request]: Include module readme updates to pipeline trigger paths

[eriqua]

Description

Due to the recently added test which verifies module readmes are compliant with the Set-ModuleReadme script, changes to the readme files should trigger the corresponding module pipelines.
In fact, changes to readme files could invalidate the test, causing the static validation step and thus the pipeline to fail.

• Update pipeline trigger paths to include module readme.
• .md files other than the module readmes should still be excluded.

[AlexanderSehr]

Not sure about this. We have the test to make sure our ReadMe's are up to date, but it won't help anybody to publish a new module version just because the readMe was updated - nor run the deployment tests.

I know where you're coming from and hence wonder if there is a way to fail GitHub pipelines on 2 levels like in Azure DevOps: Warning vs. Failed.

[AlexanderSehr]

Discussion: The test makes sense, however - we should make sure no new version is published just because a readme was changed. To that end, we should doublecheck if the Publishing (& deployment?) Logic actually excludes ReadMe files (and other non-module files, like test files) - and further update the pipeline triggers to test also ReadMe changes

[rahalan]

@MariusStorhaug , can you please have a look at the above question (publishing)?

[MariusStorhaug]

Publishing will say there is something to publish if there has been a change on a given module folder. It uses Git to find modifications or additions, finds relative folder path, finds bicep or json file, does the same for parents, then pushes those to an array with info on what to run publishing on. Talked with Erica on adding a filter to it, but thinking also that we are having some automation to cater for manual updates of the readme. How about we automate the updating when there is a PR? Creating a separate issue to discuss that topic.

[eriqua]

Moving the issue to blocked, since changes to the readme file will cause a new version of the module to be published and we agreed to not add the new trigger to the pipelines until when that won't be the case anymore.