-
Notifications
You must be signed in to change notification settings - Fork 2
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
eslint + some fixes #1
base: master
Are you sure you want to change the base?
Conversation
} | ||
|
||
me.channels[queueName].on('error', (/* err */) => { | ||
process.exit(1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Это был кодец, который попал под eslint наш.
И, соответственно, я правил тогда весь код, чтобы просто он под eslint нормально зходил. Но было это очень очень давно.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ну эт понятно, вопрос к коммиту 1a169f9 с коментом "grass from prod". Там как видишь не только форматирование поменялось, но и логика.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
а, да, там было ещё пара мест, где я фиксил именно callback hell, приводя его "безпалева" к читаемому и т.п. "работающему" состоянию, т.к. там были упущения, которые однозначно приводили к коллизиям
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ну эт понятно, вопрос к коммиту 1a169f9 с коментом "grass from prod". Там как видишь не только форматирование поменялось, но и логика.
ну это вот как раз оно, кажись, где логика была поправлена "по тихому"
Это наверно устарело?) |
не знаю, но могу сделать повторные правки в свежем коде, если он есть |
No description provided.