Add enhydris_api_client.put_timeseries() (fixes DEV-130)#60
Add enhydris_api_client.put_timeseries() (fixes DEV-130)#60aptiko merged 2 commits intoopenmeteo:masterfrom
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds a put_timeseries method to the Enhydris API client to support updating existing time series resources via HTTP PUT requests.
Key Changes:
- Implemented
put_timeseriesmethod for updating time series - Added test coverage for the new method
- Updated documentation to reflect the new update capability
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
src/enhydris_api_client/__init__.py |
Added put_timeseries method that performs PUT requests to update time series resources |
tests/enhydris_api_client/test_timeseries.py |
Added PutTimeseriesTestCase to test the new method's request behavior |
docs/enhydris_api_client.rst |
Updated documentation to include put_timeseries and clarified that methods support update operations |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #60 +/- ##
=======================================
Coverage 95.84% 95.85%
=======================================
Files 19 19
Lines 2263 2266 +3
=======================================
+ Hits 2169 2172 +3
Misses 94 94 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
4ea7fbd to
9b75d8b
Compare
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
e834b90 to
d0d0f03
Compare
d0d0f03 to
e1a166b
Compare
No description provided.