Skip to content
This repository has been archived by the owner on Mar 5, 2023. It is now read-only.

Latest commit

 

History

History
29 lines (17 loc) · 2.54 KB

CONTRIBUTING.md

File metadata and controls

29 lines (17 loc) · 2.54 KB

Contributing

There are a lot of philosphical things tied in to the way we strive to do software development. Some you are most likely familiar with, others maybe not so much.

You ain't gonna need it (YAGNI): If someone hasn't asked for it, someone's not paying for it, and it's not needed to finish what has been asked and paid for, then don't develop that thing. (There are some extreme corner cases here, but they are extreme and rare.)

Don't repeat yourself (DRY): If you are about to copy and paste that bit of code, reconsider. If you have a method that does exactly what you need it to, creating two means there's a possibility one will get out of sync or miss out on that update you make in a few months.

Keep it simple, stupid: (I could really live without the "stupid" piece.) Write the simplest code that could work. (This one actually comes with a lot of caveats, but it does hold true quite often.)

Make it SOLID: This is actually a set of principles in and of itself:

  • Single responsibility principle: Each thing you write should have one responsibility. Specifically classes.
  • Open/closed principle: Objects should be open for extension, but closed to modification.
  • Liskov substitution principle: Object should be replaceable by subtypes without affecting the correctness of the program.
  • Interface segregation principle: Prefer client-specific interfaces over generic catch-all interfaces.
  • Dependency inversion principle: Admittedly, don't really know this one well enough to say we follow it or not. (Most likely not as well as we could.)

Solutions over scapegoats: There are times when these principles will not be followed or someone will make a mistake. Berating the offender doesn't get any of us any closer to improving the execution of these principles.

Code promotion

Examples would probably do a better job of illustrating. The UserType class in laravel-registered has a field called slug. To make it easier to retrieve user objects from this field scopeWithSlug() was created in the class definition. When another class had the same field and we found ourselves about to write it again (copy and paste), we abstracted it into a trait that could be picked up by any other object.

Code access

If the language being used allows for access levels to be placed on variable, constants, methods, and so on, it is preferred that the access level begins at the most strict and becomes less restrictive out of necessity.

This reduces the size of the public API, which makes the class easier to consume by developers.