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

Feature/docsv1 #554

Merged
merged 23 commits into from
Aug 12, 2023
Merged

Feature/docsv1 #554

merged 23 commits into from
Aug 12, 2023

Conversation

amjadraza
Copy link
Contributor

@amjadraza amjadraza commented Jul 25, 2023

GPT-ENGINEER : Documentations

The first stage of Docs for get-engineer project is included in this PR. The following have been done

First Stage

  1. Set up sphinx based project documentation
  2. Generating the Package API docs using a Python Script
  3. Introduction about the Package
  4. Both .rst and .md files structures supported
  5. Re-Factored the pyproject.yml to incorporate docs dependencies.

After Merge to do

  • Generate and Host Project Documentation on readthedocs server from main branch
  • Close relevant issue.

Second Stage:

  • Re-Factor the Package code to include the docstrings
  • Add more information and tutorials on using gpt-engineer

@RareMojo thanks for your notes.

@amjadraza
Copy link
Contributor Author

@AntonOsika please see, happy to discuss if needed.

docs/DOCS_BUILDING.md Outdated Show resolved Hide resolved
@amjadraza
Copy link
Contributor Author

@AntonOsika Fixed the Links issues.

@AntonOsika
Copy link
Collaborator

AntonOsika commented Aug 9, 2023

Looks good with the ../ linking to md files etc!

I'm getting this error when running make build:
image

@@ -1,4 +1,4 @@
# These are supported funding model platforms

github: [antonosika]
patreon: gpt-engineer
patreon: user?u=2497970

Choose a reason for hiding this comment

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

Is this legit?

@amjadraza
Copy link
Contributor Author

Looks good with the ../ linking to md files etc!

I'm getting this error when running make build: image

@AntonOsika did you install the dependencies? Looks like the theme package is not installed correctly. Check if the package is installed correctly.

I have created docs using the recent push.

https://gpt-engineer.readthedocs.io/en/latest/index.html

If installing manually, use the docs/requirements.txt file for the doc dependencies.

Thanks

@AntonOsika
Copy link
Collaborator

Great, it does work with requirements.txt

How about we only put the doc requirements in requirements.txt or in pyproject.toml, not both @amjadraza ?

@AntonOsika
Copy link
Collaborator

Hey @amjadraza .

I created a new venv and verified it does work!

Let's merge. 🎉

About the requirements.txt, should we keep it do you think?

@AntonOsika AntonOsika merged commit 45db04d into gpt-engineer-org:main Aug 12, 2023
3 of 4 checks passed
70ziko pushed a commit to 70ziko/gpt-engineer that referenced this pull request Oct 25, 2023
* Initial Docs Setup

* .gitignore updated with docs

* Added Guide to Build Sphnix Docs

* Fixed typo

* Added `.readthedocs.yaml` and `docs/requirements.txt` files

* redundant extension removed

* updated `docs/requirements.txt`

* Added Respository Stats

* Added RESULTS.md of Benchmark

* Typo fixed

* Fixed local installation and typos

* Update README.md (gpt-engineer-org#561)

Fixed the route for the WINDOWS_README.md file.

* Update FUNDING.yml

* Update FUNDING.yml

* Update FUNDING.yml

* Update FUNDING.yml

* Update FUNDING.yml

* Added DISCLAIMER in docs

* Instructions to build Docs added

* added `doc` dependencies in `pyproject.toml` file using PEP621

* added command to install `docs` dependencies

* Fixed Links for `.md` files

* updated the instructions

---------

Co-authored-by: César García <cesargarciasaez@gmail.com>
Co-authored-by: Anton Osika <anton.osika@gmail.com>
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