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

Nydus image #350

Merged
merged 3 commits into from
Mar 25, 2022
Merged

Nydus image #350

merged 3 commits into from
Mar 25, 2022

Conversation

MercyMM
Copy link

@MercyMM MercyMM commented Mar 25, 2022

commit 0c0fa92 (HEAD -> gh-master-nydus-image, gh/nydus-image)
Author: gexuyang gexuyang@linux.alibaba.com
Date: Wed Mar 9 16:16:46 2022 +0800

nydus-image: add log info in diff build

Signed-off-by: gexuyang <gexuyang@linux.alibaba.com>

commit ab9cce7
Author: gexuyang gexuyang@linux.alibaba.com
Date: Wed Mar 9 15:47:49 2022 +0800

nydus-image: add log-file to set log output

Signed-off-by: gexuyang <gexuyang@linux.alibaba.com>

commit 804dc4a
Author: gexuyang gexuyang@linux.alibaba.com
Date: Wed Mar 9 15:16:34 2022 +0800

refactor: nydus-image: prepare ArgMatches in a new function

The main function is too long to understand its logic. Refactor its
ArgMatches logic by extracting it into a new function

Signed-off-by: gexuyang <gexuyang@linux.alibaba.com>

gexuyang added 3 commits March 25, 2022 14:16
The main function is too long to understand its logic. Refactor its
ArgMatches logic by extracting it into a new function

Signed-off-by: gexuyang <gexuyang@linux.alibaba.com>
Signed-off-by: gexuyang <gexuyang@linux.alibaba.com>
Signed-off-by: gexuyang <gexuyang@linux.alibaba.com>
Copy link
Collaborator

@liubogithub liubogithub left a comment

Choose a reason for hiding this comment

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

Looks OK.

@liubogithub liubogithub merged commit 4fc8f77 into dragonflyoss:master Mar 25, 2022
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.

3 participants