vladimino

{
  "repositories": [
    {
      "type": "vcs",
      "url": "https://github.com/lorenza/hello-world"
    }
  ],
  "require": {
    "lorenza/hello-world": "dev-master"
  }

vladimino

Я когда делаю Code Review критерии следующие:

• Безопасность:
  • Каждый аргумент метода простого типа должен проверяться на тип в случае его проксирования и на граничные значения в случае обработки. Чуть что не так - бросается исключение. Если метод с кучкой аргументов на 80% состоит из поверки из аргументов - это вполне норм))
  • Никаких trigger_error, только исключения.
  • Исключения ДОЛЖНЫ быть человеко-понятны, всякие "Something went wrong" можно отдавать пользователю, но в лог должно попасть исключение со стектрейсом и человеко-понятным описанием, что же там пошло не так.
  • Каждый аргумент (объект) метода должен быть с тайпхинтингом на этот его класс, или интерфейс.
  • За eval как правило шлю на **й.
  • @ допускается только в безвыходных ситуациях, например проверка json_last_error.

• Перед работой с БД - обязательная проверка данных.

vladimino

Deploying Sculpin Sites to GitHub Pages

I wanted to be able to use Sculpin to generate GitHub pages. Here's what I did...

1. Created a super awesome Sculpin site from the Sculpin Blog Skeleton

2. Make sure everything is under version control in my master branch (except things that shouldn't be. see the .gitignore)

3. Updated publish.sh:

#!/bin/bash