search

Helioviewer-Project / python-api

Python wrapper around the Helioviewer API.
https://hvpy.readthedocs.io/en/latest/
BSD 2-Clause "Simplified" License
6 stars 5 forks source link

Rename `DataSources` enum to `DataSource` #69

Closed akash5100 closed 2 years ago

akash5100 commented 2 years ago
nabobalis commented 2 years ago

I think we need to rename the file to drop the s as well.

Any place that has datasources should be singular now.

nabobalis commented 2 years ago

I think the API page will need changing?

codecov-commenter commented 2 years ago

Codecov Report

Merging #69 (6938daa) into main (c8d31f5) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #69   +/-   ##
=======================================
  Coverage   99.49%   99.49%           
=======================================
  Files          23       23           
  Lines         599      599           
=======================================
  Hits          596      596           
  Misses          3        3
Impacted Files Coverage Δ
hvpy/datasource.py 100.00% <100.00%> (ø)
hvpy/utils.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

akash5100 commented 2 years ago

Okay