-
Notifications
You must be signed in to change notification settings - Fork 441
Reflect new repo name #887
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
Conversation
This has been updated for the new github org name as well |
@@ -2,14 +2,14 @@ | |||
|
|||
This guide should teach you how to set up your development environment to start contributing to the `ilab` CLI tool. | |||
|
|||
**tl;dr** We're going to install `ilab` in a Python virtual environment like we did in the `README.md`. Instead of installing from GitHub, however, we'll clone the [`instruct-lab/cli`](https://github.com/instruct-lab/cli) repository from GitHub and install `ilab` from the cloned Python source code. | |||
**tl;dr** We're going to install `ilab` in a Python virtual environment like we did in the `README.md`. Instead of installing from GitHub, however, we'll clone the [`instructlab/instructlab`](https://github.com/instruct-lab/cli) repository from GitHub and install `ilab` from the cloned Python source code. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be (https://github.com/instructlab/cli)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch! You caught a bug in my one-liner!
git grep 'github.com/instruct-lab' | cut -f1 -d':' | sort -u | xargs sed -i -e 's/github.com\/instruct-lab/github.com\/instructlab/'
needs to be
git grep 'github.com/instruct-lab' | cut -f1 -d':' | sort -u | xargs sed -i -e 's/github.com\/instruct-lab/github.com\/instructlab/g'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be fixed!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
**tl;dr** We're going to install `ilab` in a Python virtual environment like we did in the `README.md`. Instead of installing from GitHub, however, we'll clone the [`instructlab/instructlab`](https://github.com/instruct-lab/cli) repository from GitHub and install `ilab` from the cloned Python source code. | |
**tl;dr** We're going to install `ilab` in a Python virtual environment like we did in the `README.md`. Instead of installing from GitHub, however, we'll clone the [`instructlab/instructlab`](https://github.com/instructlab/instructlab) repository from GitHub and install `instructlab` package with `ilab` command line tool from the cloned Python source code. |
The `cli` repo is now `instructlab`. Issue #817 Signed-off-by: Russell Bryant <rbryant@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thats a good point @tiran. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @russellb
commit 44d193d
Author: Russell Bryant rbryant@redhat.com
Date: Wed Apr 17 13:40:35 2024 -0400
#817