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.

Sign up for GitHub

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

Jump to bottom

Apply ternary to null operator #1110

Merged
merged 1 commit into from
Sep 21, 2023
Merged

Apply ternary to null operator #1110

merged 1 commit into from
Sep 21, 2023

Conversation

peter279k
Copy link
Contributor

@peter279k peter279k commented Sep 20, 2023 •
edited
Loading

Q A
Branch 2.x
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Documented? no

It should use the ternary null operator to process the variable is null.

New Task Checklist:

  • Are the dependencies added to the composer.json suggestions?
  • Is the doc/tasks.md file updated?
  • Are the task parameters documented?
  • Is the task registered in the tasks.yml file?
  • Does the task contains phpunit tests?
  • Is the configuration having logical allowed types?
  • Does the task run in the correct context?
  • Is the run() method readable?
  • Is the run() method using the configuration correctly?
  • Are all CI services returning green?

@veewee veewee added this to the 2.1.0 milestone Sep 21, 2023
@veewee veewee merged commit cbfbb2c into phpro:v2.x Sep 21, 2023
9 checks passed
@veewee
Copy link
Contributor

veewee commented Sep 21, 2023

Thanks! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
2.1.0
Development

Successfully merging this pull request may close these issues.
2 participants
@peter279k @veewee