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

fix typo #146

Merged
merged 5 commits into from Jan 22, 2023
Merged

fix typo #146

merged 5 commits into from Jan 22, 2023

Conversation

f4hr
Copy link
Contributor

@f4hr f4hr commented Jan 19, 2023

No description provided.

Если переданного ключа нет в передаваемом хеше, то тогда нужно вызвать блок, передав в него первым аргументом хеш, а вторым ключ:

```
env = { api_key: 123 }
setup_env(:host, env) { |env, key| env[key] = 'localhost' } # -> { api_key: 123, host: 'lockalhost' }
setup_env(:key, env) { |env, key| env[key] = 'localhost' } # -> { api_key: 123, key: 'lockalhost' }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@usernaimandrey здесь опечатка была специально сделана?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

не случайно опечатался, спасибо)


Метод fetch принемает вторым параметром значение по умалчанию:
Метод `fetch` принимает вторым параметром значение по умолчанию:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

по нашим соглашениям (сорри, щас ссылку не скину) при указании метода или функции как код мы добавляем еще скобки, вот так fetch(). Давай везде это в уроке добавим. Так студенту будут видеть, что что-то фукнция или метод.

@fey
Copy link
Contributor

fey commented Jan 21, 2023

@f4hr у вас конфликты с веткой. разрулите?

@f4hr
Copy link
Contributor Author

f4hr commented Jan 22, 2023

@f4hr у вас конфликты с веткой. разрулите?

поправил)

@fey fey merged commit 741773a into hexlet-basics:main Jan 22, 2023
@f4hr f4hr deleted the fix-typo branch January 22, 2023 13:11
@usernaimandrey usernaimandrey mentioned this pull request Jan 24, 2023
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

3 participants