Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename files tech to get-files #171

Open
blond opened this issue Jul 22, 2015 · 10 comments
Open

Rename files tech to get-files #171

blond opened this issue Jul 22, 2015 · 10 comments

Comments

@blond
Copy link
Member

blond commented Jul 22, 2015

No description provided.

@blond blond added the question label Jul 22, 2015
@arikon
Copy link
Member

arikon commented Jul 22, 2015

@blond А смысл (про все остальные задачи про переименование тот же вопрос)?

@blond
Copy link
Member Author

blond commented Jul 22, 2015

@arikon если делать #76 то совсем странным смотрится:

var bem = require('enb-bem-techs');

node.addTech(bem.levels({ levels: [] }));
node.addTech(bem.deps());
node.addTech(bem.files());

Кажется, что намного понятнее, когда похоже на методы:

var bem = require('enb-bem-techs');

node.addTech(bem.scanLevels({ levels: [] }));
node.addTech(bem.resolveDeps());
node.addTech(bem.getFiles());

@tadatuta
Copy link
Member

@blond файл может называться существительным и экспортировать метод-глагол

@blond
Copy link
Member Author

blond commented Jul 22, 2015

@tadatuta ты об этом?

var bem = require('enb-bem-techs');
var FilesTech = require('enb-bem-techs/techs/files');

node.addTech(bem.getFiles());

@tadatuta
Copy link
Member

да

@arikon
Copy link
Member

arikon commented Jul 22, 2015

@tadatuta Да ну вообще путаница будет

@blond
Copy link
Member Author

blond commented Jul 22, 2015

@tadatuta, я тоже думаю, что будет только хуже.

@arikon
Copy link
Member

arikon commented Jul 22, 2015

На мой взгляд это всё несущественно, и есть более важные вещи, чем переименование технологий

@tadatuta
Copy link
Member

На мой взгляд это всё несущественно, и есть более важные вещи, чем переименование технологий

+1

@blond
Copy link
Member Author

blond commented Jul 22, 2015

Вот так всегда :)

Ок, в 2.0 делать не будем, оставляю на подумать.

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

No branches or pull requests

4 participants