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 aliases on most commonly used tasks #247

Merged
merged 1 commit into from
Oct 18, 2023
Merged

Add aliases on most commonly used tasks #247

merged 1 commit into from
Oct 18, 2023

Conversation

HedicGuibert
Copy link
Member

This PR adds aliases on many tasks to help people annoyed by the namespaces.

In the past, we used Invoke, for which we didn't use namespaces. Migrating from Invoke to Castor makes using the namespace a bit clunky, even if they are quite handy as don't need to write them fully. Some people (including myself, as you may have guessed) are used to the old commands and would be pleased to have them back!

Since this does not change anything for the namespaces users, I think this should not be a problem.

@HedicGuibert HedicGuibert added the enhancement New feature or request label Oct 18, 2023
Copy link
Member

@lyrixx lyrixx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼 except one little comment, I like it!

.castor/infra.php Outdated Show resolved Hide resolved
@lyrixx lyrixx enabled auto-merge October 18, 2023 09:26
@lyrixx lyrixx merged commit acb26a0 into main Oct 18, 2023
3 checks passed
@lyrixx lyrixx deleted the add-some-aliases branch October 18, 2023 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants