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 to the development setup documentation #1539

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

veleek
Copy link

@veleek veleek commented Nov 21, 2022

Copy the initial section on setting up a development environment from the Core repository developer documentation. Explicitly separate the instructions for doing a manual setup from a devcontainer setup. Move the preferred devcontainer instructions to the top . Added some clarifications and cleanup to the other sections.

Proposed change

Improving developer setup documentation to make it easier for new developers to get started working with the frontend repo.

Type of change

  • Document existing features within Home Assistant
  • Document new or changing features which there is an existing pull request elsewhere
  • Spelling or grammatical corrections, or rewording for improved clarity
  • Changes to the backend of this documentation
  • Removed stale or deprecated documentation

Additional information

@veleek
Copy link
Author

veleek commented Nov 21, 2022

Note: This change pulls in a react element thingy that I copied from the core development setup page. I just assumed that this would work, but I actually don't really know.

veleek and others added 4 commits March 17, 2023 22:55
Copy the initial section on setting up a development environment from the Core repository developer documentation.
Explicitly separate the instructions for doing a manual setup from a devcontainer setup.
Move the preferred devcontainer instructions to the top .  
Added some clarifications and cleanup to the other sections.
@veleek
Copy link
Author

veleek commented Mar 18, 2023

@mrand - Rebased on master, should be good to merge now.

@mrand
Copy link

mrand commented Mar 22, 2023

@mrand - Rebased on master, should be good to merge now.

Sorry, I don't have commit access... I'm just helping out with wording. Hopefuly ludeeus can.

@veleek
Copy link
Author

veleek commented May 5, 2023

@ludeeus - Do you have permission to merge this?

@c0ffeeca7 c0ffeeca7 self-assigned this Jun 21, 2024
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.

None yet

4 participants