cognitedata / oryx

.NET Cross platform and highly composable middleware for building web request handlers in F#
Apache License 2.0
202 stars 10 forks source link

refactor: Remove pipeline abstraction #144

Closed dbrattli closed 5 months ago

dbrattli commented 7 months ago

Simplify Oryx by removing the pipeline abstraction. Oryx can now only be used for HTTP handlers. This should make the code easier to read and type hints easier to understand.

codecov[bot] commented 7 months ago

Codecov Report

Merging #144 (168f45b) into master (a521683) will decrease coverage by 0.45%. The diff coverage is 68.21%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #144 +/- ## ========================================== - Coverage 77.74% 77.29% -0.45% ========================================== Files 9 9 Lines 692 674 -18 Branches 41 40 -1 ========================================== - Hits 538 521 -17 + Misses 127 126 -1 Partials 27 27 ``` | [Files](https://app.codecov.io/gh/cognitedata/oryx/pull/144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata) | Coverage Δ | | |---|---|---| | [src/Exception.fs](https://app.codecov.io/gh/cognitedata/oryx/pull/144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-c3JjL0V4Y2VwdGlvbi5mcw==) | `14.28% <ø> (ø)` | | | [src/Fetch.fs](https://app.codecov.io/gh/cognitedata/oryx/pull/144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-c3JjL0ZldGNoLmZz) | `91.54% <ø> (ø)` | | | [src/HttpContext.fs](https://app.codecov.io/gh/cognitedata/oryx/pull/144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-c3JjL0h0dHBDb250ZXh0LmZz) | `88.00% <ø> (ø)` | | | [src/Logging.fs](https://app.codecov.io/gh/cognitedata/oryx/pull/144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-c3JjL0xvZ2dpbmcuZnM=) | `95.12% <100.00%> (ø)` | | | [src/Metrics.fs](https://app.codecov.io/gh/cognitedata/oryx/pull/144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-c3JjL01ldHJpY3MuZnM=) | `100.00% <ø> (ø)` | | | [src/Builder.fs](https://app.codecov.io/gh/cognitedata/oryx/pull/144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-c3JjL0J1aWxkZXIuZnM=) | `90.00% <50.00%> (ø)` | | | [src/Error.fs](https://app.codecov.io/gh/cognitedata/oryx/pull/144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-c3JjL0Vycm9yLmZz) | `65.04% <81.81%> (ø)` | | | [src/Core.fs](https://app.codecov.io/gh/cognitedata/oryx/pull/144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-c3JjL0NvcmUuZnM=) | `0.00% <0.00%> (ø)` | | | [src/HttpHandler.fs](https://app.codecov.io/gh/cognitedata/oryx/pull/144?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-c3JjL0h0dHBIYW5kbGVyLmZz) | `73.00% <68.18%> (ø)` | |