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

Added new composer config commands for composer authentication #12

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

sebprt
Copy link
Contributor

@sebprt sebprt commented Jun 26, 2024

No description provided.

@sebprt sebprt added the GTM Good to merge label Jun 26, 2024
@sebprt sebprt self-assigned this Jun 26, 2024
@sebprt sebprt requested a review from gplanchat June 26, 2024 14:07
@sebprt sebprt force-pushed the composer-authentication branch from e280ea7 to f1dbf98 Compare June 26, 2024 14:11
@@ -38,7 +38,7 @@ private static function escapeArgument(?string $argument): string
if (!preg_match('/[\/()%!^"<>&|\s]/', $argument)) {
return $argument;
}
$argument = preg_replace('/(\\\\+)$/', '$1$1', $argument);
$argument = preg_replace('/(\\\+)$/', '$1$1', $argument);
Copy link
Contributor

Choose a reason for hiding this comment

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

Ce changement est volontaire ou c'est CS-fixer qui l'a provoqué ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

C'est CS-Fixer qui l'a changé

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GTM Good to merge
Projects
Status: Reviewer approved
Development

Successfully merging this pull request may close these issues.

3 participants