feat: add includeEnvironment parameter #870

[massimeddu-sj]

This PR introduce a new parameter includeEnvironment that by default is true.

[monperrus]

Thanks for the PR, we'll look at it.

[monperrus]

CI is failing?

[massimeddu-sj]

CI is failing?

Apparently some tests are failing, but currently I'm not able to reproduce these failures on my local environment. I'll try to understand why they are failing and I'll try to fix them.

[massimeddu-sj]

CI is failing?

I've fixed the failing tests and the formatting issues. Now it should be good.

[massimeddu-sj]

I've rebased the PR to solve merge conflicts.

[LogFlames]

Thank you for the PR!
Please take a look at my one comment, the rest of the code looks good.
I will also look into the ˋcheck-lockfileˋ action.

[massimeddu-sj]

Thank you for the PR! Please take a look at my one comment, the rest of the code looks good. I will also look into the ˋcheck-lockfileˋ action.

Sorry, I cannot find your comment. Can you please link it or re-post here?

[LogFlames]

Of course, here it is

[massimeddu-sj]

Of course, here it is

I'm a bit confused, the link opens this same page. Maybe your comment is private or I don't have permissions to see it?

[LogFlames]

Yes you’re right, I hadn’t published the review. Now it should be visible!