8000 Remove duplication on arch detection by aliariff · Pull Request #4880 · linkerd/linkerd2 · GitHub
[go: up one dir, main page]

Skip to content

Remove duplication on arch detection - #4880

Merged
alpeb merged 2 commits into
linkerd:mainfrom
aliariff:refactor-build-cli-bin
Aug 14, 2020
Merged

alpeb merged 2 commits into
linkerd:mainfrom
aliariff:refactor-build-cli-bin

Conversation

@aliariff
Copy link
Copy Markdown
Contributor

@kleimkuhler @alpeb @cpretzer

Changes

  • Move the os architecture detection to another file, avoid duplication

@aliariff
aliariff requested a review from a team as a code owner August 13, 2020 21:06
8000
Signed-off-by: Ali Ariff <ali.ariff12@gmail.com>
@aliariff
aliariff force-pushed the refactor-build-cli-bin branch from e4d2802 to becb1cc Compare August 13, 2020 21:07
@kleimkuhler kleimkuhler left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Have a few small comments but looks good otherwise!

Comment thread bin/_os.sh Resolved
Comment thread bin/_os.sh Resolved
Comment thread bin/_os.sh Resolved
Signed-off-by: Ali Ariff <ali.ariff12@gmail.com>
@kleimkuhler kleimkuhler left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚀

@alpeb alpeb left a comment
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! 👍

@alpeb
alpeb merged commit 492b0fe into linkerd:main Aug 14, 2020
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

0