Init implementation #1
Conversation
|
Changes Unknown when pulling 51c6bde on init into * on master*. |
|
Changes Unknown when pulling 6365644 on init into * on master*. |
lib/load-compatible-agents.js
Outdated
| return got(agentsUrl) | ||
| .then(parseXMLResponse) | ||
| .then(data => { | ||
| const agents = getAgetns(data); |
| return Promise.all([ | ||
| loadBuild(buildUrl), | ||
| loadAgents(buildUrl) | ||
| ]).then(data => { |
There was a problem hiding this comment.
Нет, в ES6 .spread() не нужен, т.к. есть spread оператор:
Promise.all([...]).then([build, agents] => {});Но в 4й ноде его нет.
|
Капитально прям подошёл! |
| API | ||
| --- | ||
|
|
||
| ### queueInfo(url[, options]) |
There was a problem hiding this comment.
Это как-то автоматически генерируется?
Меня всегда смущало, что имена методов и параметров в такой доке пишутся не моноширинным шрифтом. Типа, таким образом иллюстрируя, что это код.
There was a problem hiding this comment.
Я руками пишу.
И так понятно что это про код. А использовать двойные выделения (заголовок уже выделен) плохой тон.
There was a problem hiding this comment.
Можно генерить чем-то вроде verb.
|
/ok |
|
🆙 |
|
|
||
| ```js | ||
| queueInfo('http://teamcity.domain.com', { | ||
| projectPattern: 'project :: Pull requests :: *' |
There was a problem hiding this comment.
А как правильно написать, чтобы все подпроекты учитывались? Две звёздочки?
There was a problem hiding this comment.
Одна звёздочка матчится на все подпроекты.
There was a problem hiding this comment.
Понятно, давай может добавим такой случай в пример ниже?
lib/build-filter.js
Outdated
| } | ||
|
|
||
| const hasCompatibleAgents = build.compatibleAgents && build.compatibleAgents.length !== 0; | ||
| if (options.ignoreIncompatibleAgents && build.state === 'queued' && hasCompatibleAgents) { |
There was a problem hiding this comment.
Я бы это условие и одно выше вынес в именованные функции.
There was a problem hiding this comment.
Ну, всё, что связано с анализом объекта build
| * @param {string} buildUrl - the relative url to build. | ||
| * @returns {object} | ||
| */ | ||
| function loadBuild(buildUrl) { |
There was a problem hiding this comment.
Смущает, что сюда приходит ссылка, а не buildId, например. Что ссылка формируется где-то в другом месте. Размазывание логики, кмк. Но решай сам.
There was a problem hiding this comment.
Одного buildId мало, можно целиком build передавать.
Но формирование урла зависит от переданного teamcityUrl.
|
Ok! 👌 чт, 2 июня 2016 г., 0:36 Andrew Abramov [email protected]:
|
blond
force-pushed
the
init
branch
2 times, most recently
from
1634599 to
06a6dac
Compare
|
🆙 |
blond
force-pushed
the
init
branch
2 times, most recently
from
9f8b8b4 to
259dac3
Compare
No description provided.