Added spell checker #118

dkorzhov · 2021-07-31T09:27:40Z

No description provided.

…eature/devops

Added text checker

vvssttkk · 2021-07-31T10:42:43Z

@dkorzhov спасибо
давай сделаем spell gh-action так, чтобы он не тормозил другие actions. пусть падает, но все остальное чтобы запускалось, сможешь?

dkorzhov · 2021-07-31T11:34:22Z

@dkorzhov спасибо
давай сделаем spell gh-action так, чтобы он не тормозил другие actions. пусть падает, но все остальное чтобы запускалось, сможешь?

Можно сделать continue-on-error: true и тогда он будет всегда success, но ошибку можно будет посмотреть только в логах.

Test three OS

vvssttkk

@dkorzhov спасибо
давай сделаем spell gh-action так, чтобы он не тормозил другие actions. пусть падает, но все остальное чтобы запускалось, сможешь?

Можно сделать continue-on-error: true и тогда он будет всегда success, но ошибку можно будет посмотреть только в логах.

оно отмечает зеленым job, и ошибки видно в files changed. хотелось чтобы было красным если нашло все же ошибки, но остальные запускались и не обращали внимание

dkorzhov · 2021-07-31T13:43:42Z

@dkorzhov спасибо
давай сделаем spell gh-action так, чтобы он не тормозил другие actions. пусть падает, но все остальное чтобы запускалось, сможешь?

Можно сделать continue-on-error: true и тогда он будет всегда success, но ошибку можно будет посмотреть только в логах.

оно отмечает зеленым job, и ошибки видно в files changed. хотелось чтобы было красным если нашло все же ошибки, но остальные запускались и не обращали внимание

Так вроде изначально и было такое поведение, остальные джобы выполнялись, только сам pipelien потом становился красным. Как мы хотим эти проверки использовать дальше? В отдельном файле github actions или добавим в основной?

dkorzhov · 2021-07-31T13:44:23Z

И по поводу теста на трех OS. Оставим в отдельном файле с ручным запуском? pre-commit для теста OS тогда тоже убираю?

vvssttkk · 2021-07-31T14:28:14Z

Так вроде изначально и было такое поведение, остальные джобы выполнялись, только сам pipelien потом становился красным.
сейчас ведь spell check отображается зеленым, хотя ошибки есть согласно описанию actions

Как мы хотим эти проверки использовать дальше? В отдельном файле github actions или добавим в основной?

лучше в отдельном

vvssttkk · 2021-07-31T14:29:28Z

И по поводу теста на трех OS. Оставим в отдельном файле с ручным запуском?

для этого лучше в отдельном.

pre-commit для теста OS тогда тоже убираю?

получается что да, они должны запускаться только для лекций

Updated text checker

vvssttkk · 2021-07-31T18:16:32Z

выполнить ещё локально pre-commit и сделай commit изменений, дабы посмотреть всю картину

Changed after pre-commit

SemyonSinchenko

Кажется надо довести словарь до хоть чуть-чуть разумного вида перед тем, как мержить. Мои правки, кажется, покроют самую боль про кубиты, гейты и прочее.

SemyonSinchenko · 2021-08-01T09:28:19Z

.yaspellerrc.json

+      "волонтерств.*",
+      "гайд.*",
+      "Гандапас",
+      "геймификаци.*",


Suggested change

"геймификаци.*",

"геймификаци.*",

"гейт.*",

"Гейт.*",

SemyonSinchenko · 2021-08-01T09:28:54Z

.yaspellerrc.json

+      "конфиг.*",
+      "корпоративщин.*",
+      "корутин.*",
+      "кроссфункциональн.*",


Suggested change

"кроссфункциональн.*",

"кроссфункциональн.*",

".*кубит.*",

"Кубит.*",

SemyonSinchenko · 2021-08-01T09:32:29Z

.yaspellerrc.json

+      "скилл.*",
+      "скрайбинг.*",
+      "скоммуницир.*",
+      "скоммуницируй",


Suggested change

"скоммуницируй",

"скоммуницируй",

".*скори.*",

SemyonSinchenko · 2021-08-01T09:34:19Z

.yaspellerrc.json

+      "фреймворк.*",
+      "фронтенд.*",
+      "Феншуй.*",
+      "хакатон.*",


Suggested change

"хакатон.*",

"хакатон.*",

"хардкод.*",

SemyonSinchenko · 2021-08-01T09:36:08Z

.yaspellerrc.json

+      "скоммуницир.*",
+      "скоммуницируй",
+      "скоуп.*",
+      "скринкаст.*",


Suggested change

"скринкаст.*",

"скринкаст.*",

"Слэтер.*",

SemyonSinchenko · 2021-08-01T09:38:19Z

.yaspellerrc.json

+      "Ильяхов.*",
+      "инбокс.*",
+      "инновационн.*",
+      "интранет.*",


Suggested change

"интранет.*",

"интранет.*",

"Йожи",

SemyonSinchenko · 2021-08-01T09:39:48Z

.yaspellerrc.json

+      "техрук.*",
+      "тикет.*",
+      "тимлид.*",
+      "токен.*",


Suggested change

"токен.*",

"токен.*",

"топологически",

SemyonSinchenko · 2021-08-01T09:40:51Z

.yaspellerrc.json

+      "Мавричева",
+      "Макафи",
+      "МакГрегор.*",
+      "МакКлелланда",


Suggested change

"МакКлелланда",

"МакКлелланда",

"масштабируемость",

SemyonSinchenko · 2021-08-01T09:41:29Z

.yaspellerrc.json

+      "опенсорс.*",
+      "операционк.*",
+      "отрефактори.*",
+      "оффер",


Suggested change

"оффер",

"оффер",

"пайплайн.*",

"параметризу.тся",

SemyonSinchenko · 2021-08-01T09:43:43Z

.yaspellerrc.json

+      "айтишник.*",
+      "Ана",
+      "антипаттерн.*",
+      "апдейт.*",


Suggested change

"апдейт.*",

"апдейт.*",

"аппроксиматор.*",

Moved test-psi4 to other PR

vvssttkk · 2021-08-02T16:20:34Z

@dkorzhov а можешь принять правки @SemyonSinchenko, а то вроде как они не добавились в соответствующие файлы?

vvssttkk · 2021-08-04T05:51:06Z

@dkorzhov ping

dkorzhov · 2021-08-04T06:34:54Z

@dkorzhov ping

По поводу правок. Чуть позже сегодня закину.
Я там смотрел, не совсем понятно какие забирать. Из master ветки забрать правки? Так как в этой ветке в файлы внесены правки.

vvssttkk · 2021-08-04T06:37:10Z

@dkorzhov ping

По поводу правок. Чуть позже сегодня закину.
Я там смотрел, не совсем понятно какие забирать. Из master ветки забрать правки? Так как в этой ветке в файлы внесены правки.

речь о правках @SemyonSinchenko, которые он оставил (посмотри чуть выше)

dkorzhov · 2021-08-04T06:46:17Z

@dkorzhov ping

По поводу правок. Чуть позже сегодня закину.
Я там смотрел, не совсем понятно какие забирать. Из master ветки забрать правки? Так как в этой ветке в файлы внесены правки.

речь о правках @SemyonSinchenko, которые он оставил (посмотри чуть выше)

Да, я их видел. Только не могу сообразить, как я их могу к себе забрать. Github предлагает только fetch and merge с master. Я могу их только copy-past к себе сделать, только не пойму для чего так. То есть мы спокойно можем принять PR и сделать merge в master, а от туда я заберу все изменения к себе.
Или я чего то не уловил?

SemyonSinchenko · 2021-08-04T06:49:16Z

Или я чего то не уловил?

Точно, ты же в форке... Тогда давайте мержим так, а эти правки потом отдельно

vvssttkk · 2021-08-04T08:44:46Z

@SemyonSinchenko @dkorzhov тогда льем в master?

Administrator and others added 7 commits July 31, 2021 10:14

Added checkText

c4fef8a

Merge branch 'feature/devops' of github.com:dkorzhov/qmlcourse into f…

d352531

…eature/devops

Added some checks

27e1ffa

Test ckecker

3e423a1

Test checker

c93e25d

Test checker

0ab2ec1

Merge pull request #10 from dkorzhov/feature/devops

543c518

Added text checker

dkorzhov self-assigned this Jul 31, 2021

dkorzhov mentioned this pull request Jul 31, 2021

добавить spell checker и другие прибаутки для проверки текста #104

Closed

Administrator added 3 commits July 31, 2021 12:55

Test psi4 setup

9419165

Test psi4 setup

4be5809

Test psi4 setup

0b82e63

SemyonSinchenko added ci/cd enhancement New feature or request labels Jul 31, 2021

SemyonSinchenko requested review from SemyonSinchenko and vvssttkk July 31, 2021 10:02

vvssttkk linked an issue Jul 31, 2021 that may be closed by this pull request

добавить spell checker и другие прибаутки для проверки текста #104

Closed

Administrator added 6 commits July 31, 2021 13:43

Test windows

eb8c487

Test macOS

911f1b4

Test all pipline on three OS

b52eb6e

Test continue-on-error

c261e5a

Test allow-failure

752edc6

Added fail-fast

0c290ea

Administrator and others added 2 commits July 31, 2021 14:46

Test poetry

fede26f

Merge pull request #11 from dkorzhov/feature/devops

f5448b6

Test three OS

dkorzhov mentioned this pull request Jul 31, 2021

Решить вопрос с установкой psi4 #113

Closed

vvssttkk reviewed Jul 31, 2021

View reviewed changes

Administrator and others added 3 commits July 31, 2021 20:37

Updated test checker

d76552f

Merge pull request #12 from dkorzhov/feature/devops

520d5e2

Updated text checker

Updated test psi4 on OS

9f9b2c0

vvssttkk mentioned this pull request Jul 31, 2021

Test psi4 on os #120

Merged

Administrator and others added 2 commits July 31, 2021 22:09

Changed after pre-commit

64a76dc

Merge pull request #13 from dkorzhov/feature/devops

69f080a

Changed after pre-commit

SemyonSinchenko requested changes Aug 1, 2021

View reviewed changes

Administrator and others added 2 commits August 2, 2021 18:43

Moved test-psi4 to other PR

5f7c62f

Merge pull request #14 from dkorzhov/feature/devops

d14530f

Moved test-psi4 to other PR

vvssttkk changed the base branch from test-spell-checker to master August 2, 2021 16:15

vvssttkk changed the base branch from master to test-spell-checker August 2, 2021 16:16

vvssttkk changed the title ~~Added spell checker and other~~ Added spell checker Aug 4, 2021

vvssttkk approved these changes Aug 4, 2021

View reviewed changes

vvssttkk requested a review from SemyonSinchenko August 4, 2021 07:02

SemyonSinchenko approved these changes Aug 4, 2021

View reviewed changes

vvssttkk merged commit d13b622 into quantum-ods:test-spell-checker Aug 4, 2021

vvssttkk mentioned this pull request Aug 4, 2021

added spell checker #126

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Added spell checker #118

Added spell checker #118

dkorzhov commented Jul 31, 2021

vvssttkk commented Jul 31, 2021

dkorzhov commented Jul 31, 2021

vvssttkk left a comment

dkorzhov commented Jul 31, 2021

dkorzhov commented Jul 31, 2021

vvssttkk commented Jul 31, 2021

vvssttkk commented Jul 31, 2021

vvssttkk commented Jul 31, 2021

SemyonSinchenko left a comment

SemyonSinchenko Aug 1, 2021

SemyonSinchenko Aug 1, 2021

SemyonSinchenko Aug 1, 2021

SemyonSinchenko Aug 1, 2021

SemyonSinchenko Aug 1, 2021

SemyonSinchenko Aug 1, 2021

SemyonSinchenko Aug 1, 2021

SemyonSinchenko Aug 1, 2021

SemyonSinchenko Aug 1, 2021

SemyonSinchenko Aug 1, 2021

vvssttkk commented Aug 2, 2021

vvssttkk commented Aug 4, 2021

dkorzhov commented Aug 4, 2021

vvssttkk commented Aug 4, 2021

dkorzhov commented Aug 4, 2021

SemyonSinchenko commented Aug 4, 2021

vvssttkk commented Aug 4, 2021

	"скоммуницируй",
	"скоммуницируй",
	".скори.",

	"скринкаст.*",
	"скринкаст.*",
	"Слэтер.*",

	"МакКлелланда",
	"МакКлелланда",
	"масштабируемость",

	"апдейт.*",
	"апдейт.*",
	"аппроксиматор.*",

Added spell checker #118

Added spell checker #118

Conversation

dkorzhov commented Jul 31, 2021

vvssttkk commented Jul 31, 2021

dkorzhov commented Jul 31, 2021

vvssttkk left a comment

Choose a reason for hiding this comment

dkorzhov commented Jul 31, 2021

dkorzhov commented Jul 31, 2021

vvssttkk commented Jul 31, 2021

vvssttkk commented Jul 31, 2021

vvssttkk commented Jul 31, 2021

SemyonSinchenko left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

vvssttkk commented Aug 2, 2021

vvssttkk commented Aug 4, 2021

dkorzhov commented Aug 4, 2021

vvssttkk commented Aug 4, 2021

dkorzhov commented Aug 4, 2021

SemyonSinchenko commented Aug 4, 2021

vvssttkk commented Aug 4, 2021