mistercrunch commented on code in PR #31796:
URL: https://github.com/apache/superset/pull/31796#discussion_r1920672176


##########
docker-compose.yml:
##########
@@ -89,6 +89,8 @@ services:
     restart: unless-stopped
     ports:
       - 8088:8088
+      # When in cypress-mode ->
+      - 8081:8081

Review Comment:
   One thing that's kind of good with using a different port, is that firing up 
Cypress against `docker compose up` will fail as it's looking for Superset on 
8081, forcing the user to stop docker-compose and to fire it up properly 
`CYPRES_CONFIG=true docker-compose up`, meaning it enforces having the right 
backend, with the right test sets and examples loaded up, preventing some 
confusion around using the wrong backend.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org

Reply via email to