serokell / coffer

Multi-backend password store with multiple frontends
4 stars 2 forks source link

[#89] Reuse existing megaparsec parsers #117

Closed Kariel-Myrr closed 2 years ago

Kariel-Myrr commented 2 years ago

Description

Problem:

We had a lot of repeating code in FromHttpApiData instances with CLI.Parse.

Solution:

That was fixed. Basically a lot of low parsing logic moved from CLI.Parse to CLI.Types and Coffer.Path. It was necessary to avoid cyclic dependencies.

Related issue(s)

:white_check_mark: Checklist for your Pull Request

Related changes (conditional)

Stylistic guide (mandatory)