Azure / Azure/data-api-builder
Remove isDevelopmentMode from the ISqlMetadataProvider, SqlMetadataProvider, and CosmosSqlMetadataProvider
- Dominant language
- C#
- Stars
- 1.5k
- Forks
- 370
- Avg merge
- 3d 17h
- Merged PRs (30d)
- 8
Description
We currently have a function called `IsDevelopmentMode` which is a part of the interface `ISqlMetadataProvider`. Therefore, the `SqlMetdataProvider` and the `CosmosSqlMetadataProvider` classes both implement this function.
However, all this function is doing is passing along some values that return from the `RuntimeConfigProvider`. This function is not used by the classes themselves and appears to be a case of the class handling something that is not its responsibility. We should instead favor simply using the `RuntimeConfigProvider` to determine if we are in development mode.
From the `SqlMetadataProvider`:

Contributor guide
Assessment
This issue has not been assessed yet.