Make conflict detection between branch and database less strict.

edgedb / edgedb-python

The official Python client library for EdgeDB

https://edgedb.com

Apache License 2.0

366 stars 43 forks source link

Make conflict detection between branch and database less strict. #487

Closed vpetrovykh closed 2 months ago

vpetrovykh commented 3 months ago

It is an error to specify database and branch at the same time on the same configuration level. It is OK to override previous configuration with either database or branch. It is OK for the credentials file to contain both database and branch fields in order to facilitate transitioning from old style to the new style while maintaining backwards compatibility. The values must agree, though.

aljazerzen commented 2 months ago

I've implemented the same behavior in edgedb-rust.

When writing credentials.json, I've even implemented writing both branch and database, so it is backward compatible. This is useful if credentials.json is written by CLI on the new version and then old version of bindings reads it.