another round of changes from review

Author: jemorrison

jwst/cube_build/cube_build_step.py

@@ -580,10 +580,10 @@ def check_offset_file(self):
                 " The units need to provided and only arcsec is allowed."
             )
 
-            # raise ValueError(
-            #    "Offset file is not correct. Offset file needs to have three lists: "
-            #    "filename, raoffset and decoffset all of the same length."
-            # )
+            raise ValueError(
+                "Offset file is not correct. Offset file needs to have three lists: "
+                "filename, raoffset and decoffset all of the same length."
+            ) from None
 
         offset_filename = af["filename"]
         offset_ra = af["raoffset"]

jwst/cube_build/ifu_cube.py

@@ -904,7 +904,7 @@ def build_ifucube(self):
                         nslices = 30
 
                         slicemap = [15, 14, 16, 13, 17, 12, 18, 11, 19, 10, 20, 9, 21, 8, 22, 7,
-                                    23, 6, 24, 5, 25, 4, 26, 3, 27, 2, 28, 1, 29, 0]  # fmt:skip
+                                    23, 6, 24, 5, 25, 4, 26, 3, 27, 2, 28, 1, 29, 0]  # fmt: skip
 
                         wcsobj, tr1, tr2, tr3 = nirspec._get_transforms(  # noqa: SLF001
                             input_model, np.arange(nslices)