Gylfie the Robot

Gylfie is the robot who interfere with our work process on Github and helps proper flow of issues.

Source code: https://github.com/vtambourine/robot-gylfie-webhook

Transcript

1. ГИЛЬФИ ДОТОШНЫЙ РОБОТ Вениамин Тамбурин Яндекс
3. КАЧЕСТВО КОДА • ПРОСТОЙ КОД ЛЕГКО ЧИТАТЬ • ПРОСТОЙ КОД ЛЕГКО ПИСАТЬ • В ПРОСТОМ КОДЕ ТРУДНЕЕ ДОПУСТИТЬ ОШИБКУ
4. КАЧЕСТВО КОДА • JSCS, JSHINT • PRE-COMMIT HOOKS • CODE REVIEW
7. POST PING CALL REVIEWERS ISSUE WORKFLOW payload emit
8. CALL REVIEWERS • СОЗДАТЬ ЛОКАЛЬНУЮ КОПИЮ • СПИСОК ИЗМЕНЕННЫХ ФАЙЛОВ • ОПРЕДЕЛИТЬ АВТОРОВ • ПРИЗВАТЬ РЕЦЕНЗЕНТОВ
9. CALL REVIEWERS • СОЗДАТЬ ЛОКАЛЬНУЮ КОПИЮ • СПИСОК ИЗМЕНЕННЫХ ФАЙЛОВ • ОПРЕДЕЛИТЬ АВТОРОВ • ПРИЗВАТЬ РЕЦЕНЗЕНТОВ
10. СИНХРОННЫЕ МЕТОДЫ • FS: readFileSync, writeFileSync… • CHILD PROCESS: spawSync, execSync… • REQUEST: none
11. ИЗБЕГАТЬ • ЦЕПОЧКИ КОЛБЕКОВ • САМОВЫЗЫВАЮЩИЕСЯ ФУНКЦИИ • КЛАССЫ
12. ASYNC/AWAIT
13. CALLBACK var getContributors = (callback) => { repository.getBlame((error, authors) => { if (error) callback(error) // ... callback(null, contributors) }) }
14. PROMISE var getContributors = () => { return Promise((resolve, reject) => { repository.getBlame().then((authors) => { // ... resolve(contributors) }, (error) => { reject(error) }) }) }
15. GENERATORS var getContributors = function () { return spawn(function* () { var authors = yield repository.getBlame() // ... return contributors }) }
16. SPAWN next() yields? resolve Promise true false
17. ASYNC FUNCTION async function getContributors() { var authors = await repository.getBlame() // ... return contributors }
18. ASYNC FUNCTION async function getContributors() { var authors = await repository.getBlame() // ... return contributors } getContributors().then(...)
19. ASYNC FUNCTION async handle(payload) { await cloneRepository() // … var files = await getChangedFiles() // … var authors = await getAuthors(files) // … callReviewers(contributors) }
20. TRACEUR
21. TRACEUR • ТРАНСЛЯЦИЯ КОДА НА ЛЕТУ • ПОДДЕРЖКА ASYNC/AWAIT
22. TRACEUR traceur.require.makeDefault( (filename) => { return filename.indexOf('node_modules') === -1 }, {asyncFunctions: true} )
23. TRACEUR traceur.require.makeDefault( (filename) => { return filename.indexOf('node_modules') === -1 }, {asyncFunctions: true} ) require('traceur-source-maps').install(traceur)
24. TRACEUR traceur.require.makeDefault( (filename) => { return filename.indexOf('node_modules') === -1 }, {asyncFunctions: true} ) require('traceur-source-maps').install(traceur) require('./server').default.listen(4567)
25. TRACEUR $ forever nodemon bootstrap.js