gretmn102 / gretmn102/FsharpMyExtension

refactor(Serialization.Show)!: rename `show` to `toString`

Open
#6 0 comments 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
F#
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

https://github.com/gretmn102/FsharpMyExtension/blob/0319141c7e88b4389d24067e48dab4ddb5ec5810/src/Serialization/Serializers/Show.fs#L25

## Обоснование

Сейчас функция `show` превращает `ShowS` в `string`, но название у нее неподходящее. Функция [`show` из Haskell'я](https://hackage.haskell.org/package/base-4.18.0.0/docs/Text-Show.html#v:show) именно этим и занимается. К примеру, `showList` берет список на вход, а возвращает `ShowS`; `showChar` и прочее `show`'ы примерно так и работают. По этой логике `show` должен иметь сигнатуру `'a -> ShowS`, но это не так. Функцию надо переименовать.

## Кандидаты

* `build`
* `toString`

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.