chore: Added details for running local setup#35968
Conversation
WalkthroughThe recent changes enhance the Cypress setup process by introducing interactive prompts for users during integration tests. Users can now choose whether to proceed without a local server, an accessible development URL, or to pull the latest Docker container for the TestEventDriver. This update allows for greater configurability and control over the setup process, making it more adaptable to individual user needs. Changes
Assessment against linked issues
TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add Documentation and Community
|
github-actions
bot
added
the
skip-changelog
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (1)
- contributions/ClientSetup.md (1 hunks)
Additional context used
LanguageTool
contributions/ClientSetup.md
[uncategorized] ~123-~123: Loose punctuation mark.
Context: ...eappsmith-ce:releaseimage. -yes: Selecting "yes" will skip the local s...(UNLIKELY_OPENING_PUNCTUATION)
[uncategorized] ~131-~131: Loose punctuation mark.
Context: ...:** Simply close the process. -yes: Continue on the process for next stag...(UNLIKELY_OPENING_PUNCTUATION)
[uncategorized] ~138-~138: Loose punctuation mark.
Context: ...te TED setup in local machine. -yes: Download the TED image and run the im...(UNLIKELY_OPENING_PUNCTUATION)
Additional comments not posted (3)
contributions/ClientSetup.md (3)
118-123: Improve readability of options section.The prompt and options are clear, but the formatting of the options section can be improved for better readability.
**Options:** - **`no`:** Selecting "no" will trigger the setup of a local server using Docker with the `appsmith-ce:release` image. - **`yes`:** Selecting "yes" will skip the local server setup process.Tools
LanguageTool
[uncategorized] ~123-~123: Loose punctuation mark.
Context: ...eappsmith-ce:releaseimage. -yes: Selecting "yes" will skip the local s...(UNLIKELY_OPENING_PUNCTUATION)
126-131: Improve readability of options section.The prompt and options are clear, but the formatting of the options section can be improved for better readability.
**Options:** - **`no`:** Simply close the process. - **`yes`:** Continue on the process for next stage.Tools
LanguageTool
[uncategorized] ~131-~131: Loose punctuation mark.
Context: ...:** Simply close the process. -yes: Continue on the process for next stag...(UNLIKELY_OPENING_PUNCTUATION)
133-138: Improve readability of options section.The prompt and options are clear, but the formatting of the options section can be improved for better readability.
**Options:** - **`no`:** It will not create TED setup in local machine. - **`yes`:** Download the TED image and run the image for test purposes.Tools
LanguageTool
[uncategorized] ~138-~138: Loose punctuation mark.
Context: ...te TED setup in local machine. -yes: Download the TED image and run the im...(UNLIKELY_OPENING_PUNCTUATION)
|
This PR has not seen activitiy for a while. It will be closed in 7 days unless further activity is detected. |
## Description Added more details for running local setup. <img width="730" alt="Screenshot 2024-08-29 at 12 15 09 PM" src="https://github.com/user-attachments/assets/45c63a3a-119a-4d8d-9be3-0dd9232023d7"> Fixes #`35852` ## Automation /ok-to-test tags="@tag.Sanity" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/10766942068> > Commit: b8938d1 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=10766942068&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.Sanity` > Spec: > <hr>Mon, 09 Sep 2024 05:29:13 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [x] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Enhanced the setup process for integration tests with interactive prompts, allowing users to choose options regarding local server setup and Docker container usage. - Improved user experience by providing clear choices and consequences, making the setup more configurable and informative. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
3 participants
Description
Added more details for running local setup.
Fixes #
35852Automation
/ok-to-test tags="@tag.Sanity"
🔍 Cypress test results
Tip
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/10766942068
Commit: b8938d1
Cypress dashboard.
Tags:
@tag.SanitySpec:
Mon, 09 Sep 2024 05:29:13 UTC
Communication
Should the DevRel and Marketing teams inform users about this change?
Summary by CodeRabbit