Добавил стадию документирования Swagger #63
Conversation
Менял таймаут для себя
Менял таймаут для себя
|
Класс Vrunner для вызова другой команды Oscript по сути ведь надо делать тоже самое, только вместо vrunner подставить swagger в моем случае. Может сделать какой-то универсальный класс?? |
можно получение пути к бинарю (с учетом проверки на isUnix и наличие oscript_modules) можно вынести в отделный класс со статик-методом, куда бы передавать имя бинаря. |
kuzja086
force-pushed
the
SwaggerUpdate
branch
from
78b4942 to
312e775
Compare
Вынес отдельно, но для Раннера не стал исправлять, думаю это надо отдельным реквестом, чтобы проверить все вызовы. |
|
тэкс, а может еще плагин html-publish завернуть тогда? чтобы сразу в дженкинсе можно было смотреть результат? |
kuzja086
force-pushed
the
SwaggerUpdate
branch
from
0794f27 to
380027f
Compare
|
|
||
| steps.cmd(swaggerPath + " generate --src-path $config.srcDir --out $OUT") | ||
|
|
||
| def dir = new File(OUT) |
There was a problem hiding this comment.
Нельзя использовать File. Нужно взять шаг, который бы вернул список файлов.
| steps.cmd(swaggerPath + " generate --src-path $config.srcDir --out $OUT") | ||
|
|
||
| def dir = new File(OUT) | ||
| dir.eachFile(FileType.FILES){ |
There was a problem hiding this comment.
Разве там несколько сайтов? Я не пробовал запускать ещё, но думал, что там каталог с набором файлов и единственным index.html.
Если он там один, то достаточно один раз вызвать publishHTML на каталог
There was a problem hiding this comment.
посмотрел, да, там их несколько.
раньше у publishHTML была особенность - его нельзя было вызывать несколько раз. нужно было подготовить массив из описаний публикаций и вызывать publishHTML один раз, передавая туда массив.
| Logger.println(reportdir) | ||
| Logger.println(it.path) | ||
| steps.cmd("bootprint openapi $it.path $reportdir") | ||
| publishHTML (target : [allowMissing: false, |
There was a problem hiding this comment.
Нужно обернуть вызов в steps. В контексте этого класса нет метода publishHTML
| dir.eachFile(FileType.FILES){ | ||
| Logger.println(it.name) | ||
| Logger.println(it.path) | ||
| steps.cmd("bootprint openapi $it.path $OUT_HTML") |
There was a problem hiding this comment.
Потыкал bootprint, страшненький :)
может привычный swagger-ui использовать?
https://github.com/swagger-api/swagger-ui
у него есть standalone-запуск
Closes #44 Добавил новую стадию документирования.