Skip to content
This repository has been archived by the owner on May 31, 2024. It is now read-only.

Adds Go install method for CLI #426

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

rachfop
Copy link

@rachfop rachfop commented Oct 3, 2023

Adds Go install method for CLI

Users from all platforms can use this method to install the CLI.

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

I added a new method to install the CLI.
This uses Go to install directly to the GoPath.

Tracking Issue

https://github.com/flyteorg/flyte/issues/

Follow-up issue

NA
OR
https://github.com/flyteorg/flyte/issues/

@kumare3
Copy link
Contributor

kumare3 commented Dec 12, 2023

this does not seem to work for me

@rachfop
Copy link
Author

rachfop commented Dec 12, 2023

This PR was submitted October 3, for v0.6.46

@kumare3 Looks like the monorepo migration (v0.7.6) broke the Go install method.

go install github.com/flyteorg/[email protected]
go: downloading github.com/flyteorg/flytectl v0.7.5
go install github.com/flyteorg/[email protected]
go: github.com/flyteorg/[email protected] (in github.com/flyteorg/[email protected]):
	The go.mod file for the module providing named packages contains one or
	more replace directives. It must not contain directives that would cause
	it to be interpreted differently than if it were the main module.

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