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

Pass unknown commands along to lando/ddev #41

Open
wants to merge 2 commits into
base: 2.x
Choose a base branch
from

Conversation

Yuvania
Copy link
Contributor

@Yuvania Yuvania commented Jul 31, 2024

  • Implement logic to handle unknown commands in FireApp.php.
  • Use a ternary operator in the run method to decide whether to execute a known FIRE command or pass the command to the local environment (Lando/DDEV).
  • Add the passToLocalEnv method to handle execution of commands in the local environment.

This enhancement allows FIRE to seamlessly pass commands that it doesn't recognize directly to Lando or DDEV, providing greater flexibility and reducing the need for specific command wrappers.

  • Revisar todos los alias
  • Revisar el overwrite

@Yuvania
Copy link
Contributor Author

Yuvania commented Jul 31, 2024

This is the issue: #21

@Yuvania Yuvania self-assigned this Jul 31, 2024
@rigoucr
Copy link
Contributor

rigoucr commented Aug 1, 2024

@Yuvania lets hold on this one for now , we need to first improve ddev support , specially over the vrt and xdebug commands

@rigoucr rigoucr added the enhancement New feature or request label Aug 1, 2024
@Yuvania
Copy link
Contributor Author

Yuvania commented Aug 2, 2024

@rigoucr Ok, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants