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

style: bringing project to order #223

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

denisnovikov
Copy link

@denisnovikov denisnovikov commented Oct 17, 2017

issues: #222

  • cast to order readme.md
  • cast to order contributing.md

@denisnovikov denisnovikov changed the title style: bringing project to order (closed #222) style: bringing project to order Oct 17, 2017
@dzhiriki
Copy link
Contributor

Привет.

Сорри, за долгое молчание. Этим проектом мы уже особо не занимаемся.

По поводу изменений в *.md — я не вижу никаких проблем и готов их примержить.

По поводу остальных изменений — есть пара моментов:

  • основное, что это точно лучше вынести в отдельных ПР, чтобы не было сразу разных массовых изменений в одной.
  • подключение commitizen заставляет всех использовать то же именование коммитов, что принято и в самом шаблоне (а у этого шаблона разворачивание — просто склонить репу), либо дополнительно настраивать инструментарий для того именования, что принято на этом проекте. Оба варианта не самое лучшее.
    В общем, лучше это вынести отдельно и там уже дообсудить стоит \ не стоит.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants