Satisfy pylint.

python/ctsm/crop_calendars/cropcal_module.py

@@ -443,6 +443,21 @@ def import_output(
         )
 
     # Convert time axis to integer year, saving original as 'cftime'
+    this_ds_gs = convert_time_to_int_year(filename, this_ds, this_ds_gs)
+
+    # Get number of harvests
+    this_ds_gs["NHARVESTS"] = (this_ds_gs["GDDHARV_PERHARV"] > 0).sum(dim="mxharvests")
+    # Get number of harvests that would be missed if only seeing max 1 per calendar year
+    if np.any(this_ds_gs["NHARVESTS"] > 2):
+        raise RuntimeError("How to get NHARVEST_DISCREP for NHARVESTS > 2?")
+    this_ds_gs["NHARVEST_DISCREP"] = (this_ds_gs["NHARVESTS"] == 2).astype(int)
+
+    return this_ds_gs, any_bad
+
+def convert_time_to_int_year(filename, this_ds, this_ds_gs):
+    """
+    Convert time axis to integer year, saving original as 'cftime'
+    """
     if "time_bounds" in this_ds:
         # Always true before PR #2838, when even files with all instantaneous variables got
         # time_bounds saved. After that PR (and before the segregation of instantaneous and other
@@ -462,17 +477,10 @@ def import_output(
         )
     else:
         raise RuntimeError(
-            f"{filename} seems to be neither an instantaneous file nor a combined/non-instantaneous file."
+            f"{filename} is neither an instantaneous nor a combined/non-instantaneous file."
         )
 
-    # Get number of harvests
-    this_ds_gs["NHARVESTS"] = (this_ds_gs["GDDHARV_PERHARV"] > 0).sum(dim="mxharvests")
-    # Get number of harvests that would be missed if only seeing max 1 per calendar year
-    if np.any(this_ds_gs["NHARVESTS"] > 2):
-        raise RuntimeError("How to get NHARVEST_DISCREP for NHARVESTS > 2?")
-    this_ds_gs["NHARVEST_DISCREP"] = (this_ds_gs["NHARVESTS"] == 2).astype(int)
-
-    return this_ds_gs, any_bad
+    return this_ds_gs
 
 
 def handle_zombie_crops(this_ds):