Skip to content

Conversation

russellb
Copy link
Member

@russellb russellb commented Apr 17, 2024

commit 44d193d
Author: Russell Bryant rbryant@redhat.com
Date: Wed Apr 17 13:40:35 2024 -0400

Reflect new repo name

The `cli` repo is now `instructlab`.

Issue #817

Signed-off-by: Russell Bryant <rbryant@redhat.com>

#817

@russellb
Copy link
Member Author

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.
Copy link

@oindrillac oindrillac Apr 17, 2024

Choose a reason for hiding this comment

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

Copy link
Member Author

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'

Copy link
Member Author

Choose a reason for hiding this comment

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

should be fixed!

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
**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>
Copy link
Member

@hickeyma hickeyma left a comment

Choose a reason for hiding this comment

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

Thanks for pushing the changes @russellb. Do you mind taking a look at #895 as it seems to have picked up other changes in the readme?

@tiran
Copy link
Contributor

tiran commented Apr 18, 2024

@hickeyma @russellb My PR #882 addresses a few spots that Russel missed. IMO this PR is good enough to merge it. Any large changes will cause more conflicts and work down the line. Let's get the big rename patches landed and polish afterwards.

@hickeyma
Copy link
Member

Thats a good point @tiran.

Copy link
Member

@hickeyma hickeyma left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @russellb

@hickeyma hickeyma merged commit 4150f64 into instructlab:main Apr 18, 2024
@hickeyma hickeyma deleted the new-repo-name branch April 18, 2024 10:47
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.

4 participants