opentracing / opentracing-php

OpenTracing API for PHP
Apache License 2.0
508 stars 56 forks source link

Should this package follow PSR naming convention? #69

Closed ellisv closed 4 years ago

ellisv commented 6 years ago

Doc: https://www.php-fig.org/bylaws/psr-naming-conventions/

It is common in PHP community to append Interface to the name when naming interfaces. Examples from various famous packages:

jcchavezs commented 6 years ago

I don't see any benefit on doing so. Using namespaces solves any sort of duplication issue naming we might have.

Anyways, thanks for bringing the discussion over the table.

fox91 commented 6 years ago

In my opinion, yes, the code is clearer and avoids misunderstanding. Personally I use this convention in all my projects.

For example:

// ...
use OpenTracing\Tracer as OTTracer;

final class Tracer implements OTTracer
{
    // ...

become more clearly:

// ...
use OpenTracing\TracerInterface;

final class Tracer implements TracerInterface
{
    // ...
piotrooo commented 5 years ago

This looks like completed issue.

cawolf commented 4 years ago

@jcchavezs can you close this issue?

fox91 commented 4 years ago

If I have not missed anything, it seems that the code has not been changed but the issue has been closed. Can you add the label "wontfix" to this issue?

jcchavezs commented 4 years ago

Totally @fox91. Thanks!