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

Improve the readme with contributing guidelines #7

Merged
merged 8 commits into from
Dec 15, 2024
Merged

Conversation

gagb
Copy link
Contributor

@gagb gagb commented Dec 12, 2024

Addresses issue #6

@gagb gagb requested review from afourney and Copilot December 12, 2024 23:17

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no suggestions.

@afourney
Copy link
Member

This work. It's hatch project, so I've been using:

hatch shell
and
hatch test

I'll add that when back in town.

pawarbi and others added 2 commits December 14, 2024 19:15
I have updated the readme with three changes:
- Created sections for Installation and Usage to help users
- Added installation instruction
- Added additional example of using LLM. This will be the primary use case and will help users.
@gagb gagb marked this pull request as draft December 15, 2024 02:39
@gagb
Copy link
Contributor Author

gagb commented Dec 15, 2024

I've merged suggestions from #24 and #26 here. I had to do this because for some reason CodeQL isn't running on PRs from forks --- weird. So I merged the commits from both these PRs to my branch and resolved the conflicts.

cc: @pawarbi and @haesleinhuepf, your commits and contributions are preserved in this PR. :)

@gagb gagb marked this pull request as ready for review December 15, 2024 02:53
@gagb gagb merged commit 71123a4 into main Dec 15, 2024
5 checks passed
@gagb gagb deleted the gagb/improve-readme branch December 15, 2024 02:54
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