Skip to content

переведено на компоненту v8unpack #70

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

Merged
merged 4 commits into from
May 19, 2017

Conversation

dmpas
Copy link
Member

@dmpas dmpas commented May 18, 2017

Да будут спасены наши глаза

@dmpas dmpas requested review from artbear and EvilBeaver May 18, 2017 08:28
@khorevaa
Copy link
Member

up. up. Поддерживаю

@EvilBeaver EvilBeaver merged commit 79a5e49 into oscript-library:develop May 19, 2017
@artbear
Copy link
Member

artbear commented May 19, 2017

@EvilBeaver А фичи/тесты ты прогонял?

@dmpas
Copy link
Member Author

dmpas commented May 19, 2017

@artbear я прогонял, но если честно, это было в марте. или феврале...

@EvilBeaver
Copy link
Member

@artbear нет. Я все жду когда самореализуется проект oscript-infrastructure и там все будет самопрогоняться :)

@artbear
Copy link
Member

artbear commented May 19, 2017

@artbear я прогонял, но если честно, это было в марте. или феврале...
@artbear нет. Я все жду когда самореализуется проект oscript-infrastructure и там все будет самопрогоняться :)

Да, поддержка качества у вас какая-то странная :(
Почто мержим без проверки? :(

@EvilBeaver
Copy link
Member

Ну во-первых, я доверяю @dmpas. Во-вторых, там небольшие изменения, в-третьих, я не умею прогонять тесты правильно. Есть описание для тупых: "нажми здесь"?

@dmpas
Copy link
Member Author

dmpas commented May 19, 2017

@EvilBeaver там для тестов 1С нужна, oscript-infrastructure в этом направлении вряд ли скоро самореализуется.

@EvilBeaver
Copy link
Member

@dmpas ну там до момента, когда понадобится 1С, еще три пары железных сапог стоптать надо.

@artbear
Copy link
Member

artbear commented May 19, 2017

В VSC code открываем проект и запускаем 2 таска
Testing project
и
Exec all features

смотрим результаты

@dmpas
Copy link
Member Author

dmpas commented May 19, 2017

@artbear документация есть на это? :trollface:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants