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

Add PHP file_exists #461

Merged
merged 2 commits into from
Apr 5, 2024
Merged

Add PHP file_exists #461

merged 2 commits into from
Apr 5, 2024

Conversation

erikn69
Copy link
Contributor

@erikn69 erikn69 commented Sep 29, 2022

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Description

Add PHP file_exists

@erikn69 erikn69 changed the title Patch 1 Add PHP file_exists Sep 29, 2022
@erikn69 erikn69 closed this Sep 29, 2022
@erikn69 erikn69 reopened this Sep 29, 2022
@kvz kvz enabled auto-merge (squash) April 4, 2024 20:22
@kvz
Copy link
Collaborator

kvz commented Apr 4, 2024

Sorry for the late reply! I'm okay to merge this, but can you please run yarn fix in your branch?

@erikn69
Copy link
Contributor Author

erikn69 commented Apr 4, 2024

I don't remember where or why I needed this, but I imagine I solved it by another way.
I get error locutus@2.0.29: The engine "yarn" is incompatible with this module.

@kvz
Copy link
Collaborator

kvz commented Apr 5, 2024

Ah sorry about that, I relaxed the constraints there this morning. I wonder if the lock file is gonna cause you issues you. If so, best use corepack yarn instead of yarn and I think on a recent Node.js, the correct yarn version will be used then.

auto-merge was automatically disabled April 5, 2024 16:10

Head branch was pushed to by a user without write access

@erikn69
Copy link
Contributor Author

erikn69 commented Apr 5, 2024

Done

@kvz kvz enabled auto-merge (squash) April 5, 2024 17:37
@kvz kvz merged commit 6f1f4d2 into locutusjs:main Apr 5, 2024
3 checks passed
@kvz
Copy link
Collaborator

kvz commented Apr 5, 2024

🎉 Thanks for your contribution!

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