Skip to content

Fix inconsistent switch case syntax #5

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

Merged
merged 1 commit into from
Jun 20, 2025

Conversation

theodorejb
Copy link
Contributor

This must have been a typo, since it's the only case using the non-standard syntax.

This must have been a typo, since it's the only case in the project using the non-standard syntax.
magento-devops-reposync-svc pushed a commit that referenced this pull request Dec 18, 2024
@theodorejb
Copy link
Contributor Author

@glo71317 Can this be merged?

@theodorejb
Copy link
Contributor Author

@glo71317 It's confusing since the last commit references this pull request, but it's actually an unrelated change, and this pull request was never merged. By any chance can you review this pull request?
CC @sidolov

@glo71317
Copy link
Contributor

@glo71317 It's confusing since the last commit references this pull request, but it's actually an unrelated change, and this pull request was never merged. By any chance can you review this pull request? CC @sidolov

@theodorejb will review your pull request and update.

Copy link
Contributor

@glo71317 glo71317 left a comment

Choose a reason for hiding this comment

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

switch case syntax corrected.

@glo71317
Copy link
Contributor

@sidolov Can you merge this PR?

@magento-devops-reposync-svc magento-devops-reposync-svc merged commit f163c9c into magento:main Jun 20, 2025
@glo71317
Copy link
Contributor

@theodorejb Thank you for your valuable contribution and patience throughout the process. We’ve successfully merged your changes and released a new version: v1.16.5.

@theodorejb theodorejb deleted the patch-1 branch June 20, 2025 12:34
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.

3 participants