Closed brownj85 closed 3 years ago
Merging #512 (581ee53) into master (bbae4bd) will increase coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #512 +/- ##
=======================================
Coverage 97.85% 97.85%
=======================================
Files 71 71
Lines 7406 7409 +3
=======================================
+ Hits 7247 7250 +3
Misses 159 159
Impacted Files | Coverage Δ | |
---|---|---|
strawberryfields/api/connection.py | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update bbae4bd...581ee53. Read the comment docs.
Context: The Xanadu platform API will soon be implementing versioning, allowing clients to request a particular version of the API using the 'Accept-Version' HTTP header. This header will eventually be mandatory.
Description of the Change: Adds an
api_version
str property toConnection
with the value1.0.0
. On init, adds 'Accept-Version' header toConnection._headers
with the value ofapi_version
.Benefits: Future releases of sf will be able to use the platform API once versioning becomes mandatory. Platform will be able to support older versions of sf following breaking API changes.
Possible Drawbacks: None, beyond the established drawbacks of versioning the platform API - older releases of sf will eventually be incompatible with the platform once versioning becomes mandatory. The current (non-versioned) platform API's behavior is unchanged by the 'Accept-Version' header.
Related GitHub Issues: