Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

Go module updates and cleanups #1556

Merged
merged 7 commits into from
Aug 30, 2021
Merged

Go module updates and cleanups #1556

merged 7 commits into from
Aug 30, 2021

Conversation

invidian
Copy link
Member

No description provided.

Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
Done automatically using gofumpt.

Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
They are not automatically removed by 'go mod tidy', but can be removed
using sed -i '/indirect/d' go.mod.

Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
* Move file and dir modes to named constants.
* Limit file and dir modes to only user for security.
* Add error handling for executor operations on files.

Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
Right now this message is printed from 2 places.

Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
This tests on AWS are very unstable and make CI fail very often.

As we test this feature just fine on other platforms, I think it's OK to
at least temporarily disable them on AWS, until we have more time to
investigate it.

Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
@invidian invidian requested review from surajssd, knrt10 and iaguis August 27, 2021 18:25
Copy link
Member

@knrt10 knrt10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@invidian
Copy link
Member Author

Feel free to merge :)

@knrt10 knrt10 merged commit fce27ec into master Aug 30, 2021
@knrt10 knrt10 deleted the invidian/updates branch August 30, 2021 07:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants