-
Couldn't load subscription status.
- Fork 251
Repo cleanup: consolidate/rename folders, add/update READMEs #8
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
@chriscressman
chriscressman
left a comment
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.
I took a look at all the READMEs. Looks like a good round of general cleanup to me 😄
May want to rename some of the example app directories to give them more generic names that describe the type of app or use case. The notebook directories are a good example of this.
@chriscressman
chriscressman
Jul 31, 2023
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.
This looks like something we'd want to include for all READMEs or move up higher in the hierarchy to cover everything?
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.
move up higher in the hierarchy to cover everything?
🎯
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.
Added to follow-up PR
@chriscressman
chriscressman
Jul 31, 2023
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.
We could clarify how our two docs GPT example apps relate. This one is for those who want to use the ChatGPT UI, while the other is for those who want to use the GPT API with their own UI.
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 point, I can clarify this in the individual readmes and the overall example-apps readme.
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.
Added to follow-up PR
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.
looks great! could you rebase?
Uh oh!
There was an error while loading. Please reload this page.
i️ Don't worry about the files changed number, it's ~90% renaming 😄
Probably didn't catch everything here, but can circle back for smaller, targeted updates.