fix: overwrite params with environment variables even if there are no params in the pipeline definition; make mypy ignore REST API tests

[anakin87]

Related Issues

• fixes KeyError thrown with Docker Compose if no params for DocumentStore is defined #3676

Proposed Changes:

Before

haystack/haystack/pipelines/config.py

Line 82 in 5c53b2b

component_definition["params"][param_name] = value

I check whether the key params exists. If not, I create it.

How did you test it?

Added a new test in the REST API tests.
I manually verified that it also works in Docker.

Notes for the reviewer

Mypy failed because it checked the REST API tests.
These files should be ignored, as done in #3894 for main tests.
So I also fixed this aspect.

Checklist

• I have read the contributors guidelines and the code of conduct
• I have updated the related issue with new insights and changes
• I added tests that demonstrate the correct behavior of the change
• I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.
• I documented my code
• I ran pre-commit hooks and fixed any issue

[anakin87]

@ZanSara @masci
tests failures seem not related to this PR. 🤔

[anakin87]

Adapted this PR after the merge of #3956.

I also made some little fixes to the action:

• search for changed python files also in non-root directories
• use the right id to recover the output (files instead of python-files)
• use steps.files.outputs.all_changed_files instead of steps.files.outputs.changed_files to get the changed files

[masci]

This looks great, thanks!

[masci]

Arf, good catch!