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

feat: support docker language #67

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

feat: support docker language #67

wants to merge 16 commits into from

Conversation

bxb100
Copy link

@bxb100 bxb100 commented Nov 15, 2024

close: #49

TODO

  • add test suits
  • test inner container
    • disable MacOS test, because it's without docker1
    • disable Windows test, because it's without linux kernel2
  • test windows

Footnotes

  1. https://github.com/actions/runner-images/issues/17

  2. https://github.com/orgs/community/discussions/21317

Cargo.toml Outdated Show resolved Hide resolved
@j178
Copy link
Owner

j178 commented Nov 17, 2024

Thanks for your fantastic work! I'll review it soon!

@bxb100 bxb100 marked this pull request as ready for review November 17, 2024 11:58
@bxb100 bxb100 marked this pull request as draft November 17, 2024 15:24
@bxb100
Copy link
Author

bxb100 commented Nov 17, 2024

@j178 I want to rewrite all commands to this crate https://github.com/fussybeaver/bollard , so maybe late review /= =/

@j178
Copy link
Owner

j178 commented Nov 17, 2024

@bxb100 I would prefer using the Docker CLI because it is lightweight and doesn't require many dependencies for basic Docker operations, unlike bollard, which seems a bit heavy to me. What do you think?

@bxb100
Copy link
Author

bxb100 commented Nov 17, 2024

@j178 Ok

@bxb100 bxb100 marked this pull request as ready for review November 17, 2024 16:33
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.

Support docker language
2 participants