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

docs: move docs to https://docs.espressif.com/projects/vscode-esp-idf-extension/en/latest/ #1226

Merged
merged 7 commits into from
Jun 25, 2024

Conversation

kolipakakondal
Copy link
Collaborator

Description

Please include a summary of the change and which issue is fixed.

Fixes #XXX

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Steps to test this pull request

Provide a list of steps to test changes in this PR and required output

  1. Click on "Command"
  2. Execute action.
  3. Observe results.
  • Expected behaviour:

  • Expected output:

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

@kolipakakondal kolipakakondal changed the title Docs espressif docs: move docs to https://docs.espressif.com/projects/vscode-esp-idf-extension/en/latest/ Jun 23, 2024
@kolipakakondal
Copy link
Collaborator Author

Hi @brianignacio5 @radurentea Preview docs published here for vscode https://preview-docs.espressif.com/projects/vscode-esp-idf-extension/en/1226-merge/index.html

PR contains only docs structure and CI integration, you can merge this and take over for the rest of the documentation.

Once we everything is done, will publish to the production docs.

Copy link
Collaborator

@brianignacio5 brianignacio5 left a comment

Choose a reason for hiding this comment

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

LGTM

Just need to add docs_espressif to be ignored by linter by adding it to .prettierignore file

@kolipakakondal kolipakakondal merged commit 4ea0963 into master Jun 25, 2024
5 of 7 checks passed
@kolipakakondal kolipakakondal deleted the docs_espressif branch June 25, 2024 15:38
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.

2 participants