-
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
docs: add passive option disclaimer #48
docs: add passive option disclaimer #48
Conversation
ff15d9d
to
1a38a00
Compare
✅ Successfully deployed static |
[hermione-passive-browsers](https://github.com/gemini-testing/testplane-passive-browsers). | ||
Доступна с [email protected]. | ||
|
||
В данный момент эта опция не работает с html-reporter. В данный момент мы рекомендуем продолжить использовать плагин [hermione-passive-browsers](https://github.com/gemini-testing/testplane-passive-browsers). Обратите внимание, что нельзя использовать плагин и эту опцию одновременно. Мы уже работаем над [исправлением](https://github.com/gemini-testing/testplane/pull/1001) проблемы. |
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.
Зачем фиксироваться на конкретный PR?
Давай просто напишем, что работаем над исправлением
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.
Ну фикс же будет в рамках этого PR? Как иначе пользователям можно отслеживать прогресс по починке?
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.
Ну фикс же будет в рамках этого PR?
Не факт. Тот подход просто не сработал на фундаментальном уровне, там не "что-то доработать надо", там нужен новый PR
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.
По этому PR можно было бы выйти и на финальное решение через связи. Имхо странно писать "уже работаем" и не указывать ничего для отслеживания работы. Но если критично, ок, убрал ссылку.
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.
Да "оставлять ссылку на аппрувнутый PR, который просто не мержат" уже кучу времени - тоже как-то странно.
В моем представлении, PR больше для контрибьюторов. Для пользователей существуют issue/проекты
1a38a00
to
f294b59
Compare
No description provided.