Deprecate FileList methods #290
Conversation
|
Нужно, чтобы пользователь увидел сообщение в консоле при сборке, если будет использовать данный модуль. И ещё нужно явно говорить, что модуль будет удалён в версии |
|
Не нашла примера для прокидывания в консоль сообщений о deprecated-методах. |
|
Кажется, что тут лучше кидать сообщение, когда метод вызывается. Во-первых так больше засрётся консоль и шанс, что увидят больше. Во-вторых, те кто не используют этих методов не должны видеть варнингов. |
|
@blond а на приватный метод тоже нужно это сообщение? |
|
Нет, только на публичные. |
| console.log('Use bem-methods!!!'); | ||
|
|
||
| return parseFilename; | ||
| }, |
There was a problem hiding this comment.
@blond так правильно делать?
There was a problem hiding this comment.
Нет. Раньше функция на вход принимала строку, а на выход объект.
Из-за такого изменения, функция всегда будет возвращать функцию.
| console.log('WARN!!!'); | ||
| console.log('`loadFromDirSync` is deprecated!!!'); | ||
| console.log('It will be removed in v1.0.0!!!'); | ||
| console.log('Use bem-methods!!!'); |
There was a problem hiding this comment.
Что такое bem-methods? Лучше ничего не писать про что нужно использовать взамен.
Close #268