Skip to content
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

Update test.js #132

Closed
wants to merge 1 commit into from
Closed

Update test.js #132

wants to merge 1 commit into from

Conversation

sviat-o-slav
Copy link
Contributor

Different values in task (codepage Windows-1251) and tests (check for UTF-8)!

Different values in task (codepage Windows-1251) and tests (check for UTF-8)!
@fey
Copy link
Contributor

fey commented Apr 4, 2022

@battiw добрый день.
Тесты связаны с эталонным решением, т.е. если вы меняете тест, то должны изменить и решение. Но, мне кажется, здесь дело в условии задания. Нужно спросить @Psixodelik по задумке

@fey fey requested a review from Psixodelik April 4, 2022 08:14
@Psixodelik
Copy link
Contributor

Ой, тут действительно надо не только тест изменить, но и эталонное решение, иначе тесты не проходят. Добавьте исправления в index.html этого урока, чтобы там тоже кодировка была windows-1251 и тогда можно принимать. Спасибо за нахождения нестыковки :)

@battiw
Copy link
Contributor

battiw commented Apr 4, 2022 via email

@fey
Copy link
Contributor

fey commented Apr 4, 2022

@battiw вам нужно поправить еще index.html, иначе изменения нелльзя будет принять

@sviat-o-slav sviat-o-slav deleted the patch-1 branch April 4, 2022 15:24
@fey
Copy link
Contributor

fey commented Apr 4, 2022

ой извините, я всех напутал :D

@battiw
Copy link
Contributor

battiw commented Oct 11, 2022 via email

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

Successfully merging this pull request may close these issues.

None yet

4 participants