cloudflare / cloudflare/pingora

Make serde_yaml optional or configurable

Open
#779 1 comment 0 reactions 1 assignee Claimed by @johnhurt View on GitHub
dependencies
Dominant language
Rust
Stars
27.4k
Forks
1.7k
Avg merge
6h 22m
Merged PRs (30d)
3

Description

## What is the problem your feature solves, or the need it fulfills?

`serde-yaml` has been deprecated (https://github.com/dtolnay/serde-yaml). Many projects are moving to maintained alternatives -- [`serde_norway`](https://docs.rs/serde_norway/latest/serde_norway/)] or [`serde_yaml_ng`](https://docs.rs/serde_yaml_ng/latest/serde_yaml_ng/).

It would be nice if the YAML dependencies in pingora were either optional (behind a feature flag) or configurable. This would allow projects that have upgraded to remove the unmaintained depedency from their transitive dependencies to either omit YAML support or choose to use the same package they have moved to.

## Describe the solution you'd like

Make `serde_yaml` dependency optional (with a `yaml` feature) and/or add optional dependencies for other libraries.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.