Skip to content

Commit

Permalink
Depends were never actually required
Browse files Browse the repository at this point in the history
It was listed in that `requiredFields` array, but was being defaulted to
the empty string elsewhere. There is a test that confirms its optional,
which makes sense anyway.
  • Loading branch information
pbrisbin committed Jan 3, 2023
1 parent 9667923 commit 25127fc
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Database/Schema/Migrations/Filesystem.hs
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ data MigrationYaml = MigrationYaml
, myDescription :: Maybe Text
, myApply :: Text
, myRevert :: Maybe Text
, myDepends :: Text
, myDepends :: Maybe Text
-- ^ White-space separated names
}
deriving Generic
Expand All @@ -139,7 +139,7 @@ migrationYamlToMigration theId my = Migration
, mDesc = myDescription my
, mApply = myApply my
, mRevert = myRevert my
, mDeps = T.words $ myDepends my
, mDeps = maybe [] T.words $ myDepends my
}

newtype UTCTimeYaml = UTCTimeYaml
Expand Down

0 comments on commit 25127fc

Please sign in to comment.