Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: sort volume args in DOCKER_COMPAT mode #417

Merged
merged 1 commit into from
May 26, 2023
Merged

Conversation

ningziwen
Copy link
Member

@ningziwen ningziwen commented May 26, 2023

Issue #, if available:
#418

Description of changes:
Sort volume args in DOCKER_COMPAT mode

Testing done:
Unit tests and new e2e tests. runfinch/common-tests#66

  • [ X ] I've reviewed the guidance in CONTRIBUTING.md

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@ningziwen ningziwen force-pushed the sort branch 4 times, most recently from 5b4e214 to 66f0e6d Compare May 26, 2023 08:13
@ningziwen ningziwen requested review from pendo324 and vsiravar May 26, 2023 08:16
.github/workflows/ci.yaml Show resolved Hide resolved
cmd/finch/nerdctl.go Outdated Show resolved Hide resolved
austinvazquez
austinvazquez previously approved these changes May 26, 2023
Copy link
Member

@austinvazquez austinvazquez left a comment

Choose a reason for hiding this comment

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

LGTM, only had minor feedback; not really worth another revision imo though.

cmd/finch/nerdctl.go Show resolved Hide resolved
cmd/finch/nerdctl.go Show resolved Hide resolved
cmd/finch/nerdctl.go Outdated Show resolved Hide resolved
cmd/finch/nerdctl.go Outdated Show resolved Hide resolved
cmd/finch/nerdctl.go Outdated Show resolved Hide resolved
cmd/finch/nerdctl.go Show resolved Hide resolved
cmd/finch/nerdctl.go Outdated Show resolved Hide resolved
monirul
monirul previously approved these changes May 26, 2023
Copy link
Contributor

@monirul monirul left a comment

Choose a reason for hiding this comment

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

Approved with some suggestion.

cmd/finch/nerdctl.go Show resolved Hide resolved
cmd/finch/nerdctl.go Show resolved Hide resolved
pendo324
pendo324 previously approved these changes May 26, 2023
Signed-off-by: Ziwen Ning <ningziwe@amazon.com>
Copy link
Contributor

@vsiravar vsiravar left a comment

Choose a reason for hiding this comment

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

LGTM

@ningziwen ningziwen enabled auto-merge (squash) May 26, 2023 20:54
@ningziwen ningziwen merged commit 6a8ca1a into runfinch:main May 26, 2023
vsiravar pushed a commit that referenced this pull request Jun 2, 2023
🤖 I have created a release *beep* *boop*
---


## [0.6.2](v0.6.1...v0.6.2)
(2023-06-02)


### Bug Fixes

* sort volume args in DOCKER_COMPAT mode
([#417](#417))
([6a8ca1a](6a8ca1a))
* Stop lima process in postinstall and uninstall
([#438](#438))
([975c4ee](975c4ee))


### Build System or External Dependencies

* **deps:** Bump github.com/docker/docker from 23.0.6+incompatible to
24.0.0+incompatible
([#406](#406))
([25aa324](25aa324))
* **deps:** Bump github.com/docker/docker from 24.0.0+incompatible to
24.0.1+incompatible
([#413](#413))
([bb1cab2](bb1cab2))
* **deps:** Bump github.com/docker/docker from 24.0.1+incompatible to
24.0.2+incompatible
([#419](#419))
([f6f8aeb](f6f8aeb))
* **deps:** Bump github.com/lima-vm/lima from 0.15.1 to 0.16.0
([#428](#428))
([6a67f0c](6a67f0c))
* **deps:** Bump github.com/onsi/ginkgo/v2 from 2.9.5 to 2.9.7
([#432](#432))
([0a381f1](0a381f1))
* **deps:** Bump github.com/onsi/gomega from 1.27.6 to 1.27.7
([#409](#409))
([80513c7](80513c7))
* **deps:** Bump github.com/runfinch/common-tests from 0.6.5 to 0.7.0
([#420](#420))
([ac18046](ac18046))
* **deps:** Bump github.com/shirou/gopsutil/v3 from 3.23.4 to 3.23.5
([#437](#437))
([8635294](8635294))
* **deps:** Bump github.com/sirupsen/logrus from 1.9.0 to 1.9.1
([#405](#405))
([05b6b2e](05b6b2e))
* **deps:** Bump github.com/sirupsen/logrus from 1.9.1 to 1.9.2
([#408](#408))
([0ff477a](0ff477a))
* **deps:** Bump github.com/stretchr/testify from 1.8.2 to 1.8.4
([#429](#429))
([0f7750e](0f7750e))
* **deps:** Bump golang.org/x/tools from 0.9.1 to 0.9.3
([#443](#443))
([b192874](b192874))
* **deps:** Bump k8s.io/apimachinery from 0.27.1 to 0.27.2
([#407](#407))
([e6cbd2c](e6cbd2c))
* **deps:** Bump submodules
([#402](#402))
([06d86b2](06d86b2))
* **deps:** Bump submodules
([#439](#439))
([e9551b5](e9551b5))

---
This PR was generated with [Release
Please](/~https://github.com/googleapis/release-please). See
[documentation](/~https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants