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

update mftf section in readme #913

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

piotrkwiecinski
Copy link
Contributor

selenium configuration sits in compose.yaml instead of compose.dev.yaml

Closes #867

selenium configuration sits in compose.yaml instead of compose.dev.yaml
@what-the-diff
Copy link

what-the-diff bot commented Apr 20, 2023

PR Summary

  • Rename compose.dev.yaml file to compose.yaml
    This update simplifies the file handling for future development.
  • Update references in bin/mftf script
    All references are updated to match the new file name.
  • Add SELENIUM_HOST line in the .env file
    This improves the Selenium integration with Magento and ensures tests run smoothly.
  • Enable access to the dev section in Nginx config
    This allows for successful execution of MFTF tests without permission-related issues.

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.

Can't enable Selenium in compose.dev.yaml file
1 participant