Skip to content

Commit 25127fc

Browse files
committed
Depends were never actually required
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.
1 parent 9667923 commit 25127fc

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/Database/Schema/Migrations/Filesystem.hs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ data MigrationYaml = MigrationYaml
113113
, myDescription :: Maybe Text
114114
, myApply :: Text
115115
, myRevert :: Maybe Text
116-
, myDepends :: Text
116+
, myDepends :: Maybe Text
117117
-- ^ White-space separated names
118118
}
119119
deriving Generic
@@ -139,7 +139,7 @@ migrationYamlToMigration theId my = Migration
139139
, mDesc = myDescription my
140140
, mApply = myApply my
141141
, mRevert = myRevert my
142-
, mDeps = T.words $ myDepends my
142+
, mDeps = maybe [] T.words $ myDepends my
143143
}
144144

145145
newtype UTCTimeYaml = UTCTimeYaml

0 commit comments

Comments
 (0)