Skip to content

Instantly share code, notes, and snippets.

View vladimino's full-sized avatar
💙
Spreading positive vibes

Vlad Bilokur vladimino

💙
Spreading positive vibes
View GitHub Profile
@vladimino
vladimino / README.md
Created November 22, 2015 22:39 — forked from mrkrstphr/README.md
Deploying Sculpin Sites to GitHub Pages

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

@vladimino
vladimino / CodeReview.md
Last active April 22, 2017 17:05
Code Review Rules

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

  • Безопасность:
    • Каждый аргумент метода простого типа должен проверяться на тип в случае его проксирования и на граничные значения в случае обработки. Чуть что не так - бросается исключение. Если метод с кучкой аргументов на 80% состоит из поверки из аргументов - это вполне норм))
    • Никаких trigger_error, только исключения.
    • Исключения ДОЛЖНЫ быть человеко-понятны, всякие "Something went wrong" можно отдавать пользователю, но в лог должно попасть исключение со стектрейсом и человеко-понятным описанием, что же там пошло не так.
    • Каждый аргумент (объект) метода должен быть с тайпхинтингом на этот его класс, или интерфейс.
    • За eval как правило шлю на **й.
    • @ допускается только в безвыходных ситуациях, например проверка json_last_error.
  • Перед работой с БД - обязательная проверка данных.
@vladimino
vladimino / composer.json
Last active February 19, 2019 10:21 — forked from andyshinn/composer.json
Docker Compose PHP Composer Example
{
"repositories": [
{
"type": "vcs",
"url": "https://github.com/lorenza/hello-world"
}
],
"require": {
"lorenza/hello-world": "dev-master"
}