Skip to content

Commit 7c0a91e

Browse files
committed
Disable fixme lint; these are well-documented
1 parent 6d6c6a5 commit 7c0a91e

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

_delphi_utils_python/delphi_utils/validator/datafetcher.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,9 @@ def get_geo_signal_combos(data_source):
111111
Cross references based on combinations reported available by COVIDcast metadata.
112112
"""
113113
# Maps data_source name with what's in the API, lists used in case of multiple names
114-
# TODO: https://github.com/cmu-delphi/covidcast-indicators/issues/1457
115-
# Extract this mapping from meta response instead of hard-coding
114+
# pylint: disable=fixme
115+
# TODO: Extract this mapping from meta response instead of hard-coding
116+
# https://github.com/cmu-delphi/covidcast-indicators/issues/1457
116117
source_signal_mappings = {
117118
'indicator-combination': ['indicator-combination-cases-deaths'],
118119
'quidel': ['quidel-covid-ag'],

_delphi_utils_python/tests/validator/test_datafetcher.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,9 @@ def test_make_date_filter(self):
2121
assert not date_filter(FILENAME_REGEX.match("20200620_a_b.csv"))
2222
assert not date_filter(FILENAME_REGEX.match("202006_a_b.csv"))
2323

24-
# TODO: https://github.com/cmu-delphi/covidcast-indicators/issues/1456
25-
# mock out the advanced meta endpoint /covidcast/meta as well
24+
# pylint: disable=fixme
25+
# TODO: mock out the advanced meta endpoint /covidcast/meta as well
26+
# https://github.com/cmu-delphi/covidcast-indicators/issues/1456
2627
@mock.patch("covidcast.metadata")
2728
def test_get_geo_signal_combos(self, mock_metadata):
2829
"""Test that the geo signal combos are correctly pulled from the covidcast metadata."""

0 commit comments

Comments
 (0)