Skip to content

Check is_dir(WP_CONTENT_DIR) or codecept_root_dir(WP_CONTENT_DIR) #772

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

Conversation

BrianHenryIE
Copy link
Contributor

#745 requested allowing WP_CONTENT_DIR in config. #750 implemented it for absolute paths. This PR checks does the path provided exist, and if not it attempts to treat it as a relative path.

Basically, I moved my project and it took me a while to figure out why tests weren't running anymore.

Copy link

This PR is stale because it has been open 20 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@lucatume
Copy link
Owner

lucatume commented Mar 26, 2025

Sorry @BrianHenryIE; this had fell off my radar. I'm solving a build issue first, then dealing with this.

Copy link

This PR is stale because it has been open 20 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Apr 16, 2025
@lucatume lucatume removed the Stale label Apr 23, 2025
@lucatume
Copy link
Owner

Finally fixed the build, I will be able to take a look at this and add required tests.

@lucatume lucatume merged commit d70d9a1 into lucatume:master Apr 25, 2025
58 checks passed
@lucatume
Copy link
Owner

Thank you @BrianHenryIE for the PR, merged and out in 4.5.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants