From 9a8de773c7edacdb0c0179b41d82bfcf8ec26617 Mon Sep 17 00:00:00 2001 From: Brad Reed Date: Thu, 6 Mar 2025 21:12:34 +0700 Subject: [PATCH 1/5] migrate config --- README.md | 38 ++++---- core/src/Config.purs | 4 +- src/Spago/Command/Init.purs | 10 +-- src/Spago/Config.js | 25 ++++-- src/Spago/Psa.purs | 2 +- .../build/migrate-config/migrated-spago.yaml | 2 +- test/Prelude.purs | 90 ++++++++----------- test/Spago/Build.purs | 6 +- test/Spago/Build/Pedantic.purs | 7 +- test/Spago/Config.purs | 2 +- test/Spago/Test.purs | 4 +- 11 files changed, 94 insertions(+), 96 deletions(-) diff --git a/README.md b/README.md index 38b21275f..55aec7680 100644 --- a/README.md +++ b/README.md @@ -1448,6 +1448,26 @@ package: # see https://yaml-multiline.info/ - byPrefix: > "Data.Map"'s `Semigroup instance` + + # Specify whether to censor warnings coming from the compiler + # for files from this package's test code. + # Optional and can be one of two possible values + censorTestWarnings: + # Value 1: "all" - All warnings are censored + all + + # Value 2: `NonEmptyArray (Either String { byPrefix :: String })` + # - String values: + # censor warnings if the code matches this code + # - { byPrefix } values: + # censor warnings if the warning's message + # starts with the given text + - CodeName + # Note: when using `byPrefix`, use the `>` for block-string: + # see https://yaml-multiline.info/ + - byPrefix: > + "Data.Map"'s `Semigroup instance` + # Convert compiler warnings for files in this package's src code # into errors that can fail the build. # Optional and defaults to false @@ -1498,24 +1518,6 @@ package: # Optional boolean that defaults to `false`. pedanticPackages: false - # Specify whether to censor warnings coming from the compiler - # for files from this package's test code. - # Optional and can be one of two possible values - censorTestWarnings: - # Value 1: "all" - All warnings are censored - all - - # Value 2: `NonEmptyArray (Either String { byPrefix :: String })` - # - String values: - # censor warnings if the code matches this code - # - { byPrefix } values: - # censor warnings if the warning's message - # starts with the given text - - CodeName - # Note: when using `byPrefix`, use the `>` for block-string: - # see https://yaml-multiline.info/ - - byPrefix: > - "Data.Map"'s `Semigroup instance` # Convert compiler warnings for files from this package's test code # into errors that can fail the build. # Optional and defaults to false diff --git a/core/src/Config.purs b/core/src/Config.purs index 5e9339243..dd80e1aca 100644 --- a/core/src/Config.purs +++ b/core/src/Config.purs @@ -168,7 +168,6 @@ type TestConfig = { main :: String , execArgs :: Maybe (Array String) , dependencies :: Dependencies - , censorTestWarnings :: Maybe CensorBuildWarnings , strict :: Maybe Boolean , pedanticPackages :: Maybe Boolean } @@ -177,7 +176,6 @@ testConfigCodec :: CJ.Codec TestConfig testConfigCodec = CJ.named "TestConfig" $ CJS.objectStrict $ CJS.recordProp @"main" CJ.string $ CJS.recordPropOptional @"execArgs" (CJ.array CJ.string) - $ CJS.recordPropOptional @"censorTestWarnings" censorBuildWarningsCodec $ CJS.recordPropOptional @"strict" CJ.boolean $ CJS.recordPropOptional @"pedanticPackages" CJ.boolean $ CJS.recordProp @"dependencies" dependenciesCodec @@ -196,6 +194,7 @@ backendConfigCodec = CJ.named "BackendConfig" $ CJS.objectStrict type PackageBuildOptionsInput = { censorProjectWarnings :: Maybe CensorBuildWarnings + , censorTestWarnings :: Maybe CensorBuildWarnings , strict :: Maybe Boolean , pedanticPackages :: Maybe Boolean } @@ -203,6 +202,7 @@ type PackageBuildOptionsInput = packageBuildOptionsCodec :: CJ.Codec PackageBuildOptionsInput packageBuildOptionsCodec = CJ.named "PackageBuildOptionsInput" $ CJS.objectStrict $ CJS.recordPropOptional @"censorProjectWarnings" censorBuildWarningsCodec + $ CJS.recordPropOptional @"censorTestWarnings" censorBuildWarningsCodec $ CJS.recordPropOptional @"strict" CJ.boolean $ CJS.recordPropOptional @"pedanticPackages" CJ.boolean $ CJS.record diff --git a/src/Spago/Command/Init.purs b/src/Spago/Command/Init.purs index 72a97a22a..820ecbf53 100644 --- a/src/Spago/Command/Init.purs +++ b/src/Spago/Command/Init.purs @@ -159,7 +159,7 @@ defaultConfig { name, withWorkspace, testModuleName } = do pkg = { name , dependencies: [ "effect", "console", "prelude" ] - , test: Just { moduleMain: testModuleName, strict: Nothing, censorTestWarnings: Nothing, pedanticPackages: Nothing, dependencies: Nothing } + , test: Just { moduleMain: testModuleName, strict: Nothing, pedanticPackages: Nothing, dependencies: Nothing } , build: Nothing } defaultConfig' case withWorkspace of @@ -173,7 +173,6 @@ type DefaultConfigPackageOptions = Maybe { moduleMain :: String , strict :: Maybe Boolean - , censorTestWarnings :: Maybe Config.CensorBuildWarnings , pedanticPackages :: Maybe Boolean , dependencies :: Maybe Config.Dependencies } @@ -181,6 +180,7 @@ type DefaultConfigPackageOptions = Maybe { strict :: Maybe Boolean , censorProjectWarnings :: Maybe Config.CensorBuildWarnings + , censorTestWarnings :: Maybe Config.CensorBuildWarnings , pedanticPackages :: Maybe Boolean } } @@ -212,17 +212,17 @@ defaultConfig' opts = { name , dependencies: Dependencies $ Map.fromFoldable $ map mkDep dependencies , description: Nothing - , build: build <#> \{ censorProjectWarnings, strict, pedanticPackages } -> + , build: build <#> \{ censorProjectWarnings, censorTestWarnings, strict, pedanticPackages } -> { censorProjectWarnings + , censorTestWarnings , strict , pedanticPackages } , run: Nothing - , test: test <#> \{ moduleMain, censorTestWarnings, strict, pedanticPackages, dependencies: testDeps } -> + , test: test <#> \{ moduleMain, strict, pedanticPackages, dependencies: testDeps } -> { dependencies: fromMaybe (Dependencies Map.empty) testDeps , execArgs: Nothing , main: moduleMain - , censorTestWarnings , strict , pedanticPackages } diff --git a/src/Spago/Config.js b/src/Spago/Config.js index 722316ed6..75dffa049 100644 --- a/src/Spago/Config.js +++ b/src/Spago/Config.js @@ -5,17 +5,21 @@ const getOrElse = (node, key, fallback) => { node.set(key, fallback); } return node.get(key); -} +}; export function addPackagesToConfigImpl(doc, isTest, newPkgs) { const pkg = doc.get("package"); const deps = (() => { if (isTest) { - const test = getOrElse(pkg, "test", doc.createNode({ main: "Test.Main", dependencies: [] })); + const test = getOrElse( + pkg, + "test", + doc.createNode({ main: "Test.Main", dependencies: [] }), + ); return getOrElse(test, "dependencies", doc.createNode([])); } else { - return getOrElse(pkg, "dependencies", doc.createNode([])) + return getOrElse(pkg, "dependencies", doc.createNode([])); } })(); @@ -59,7 +63,9 @@ export function addPackagesToConfigImpl(doc, isTest, newPkgs) { export function removePackagesFromConfigImpl(doc, isTest, shouldRemove) { const pkg = doc.get("package"); - const deps = isTest ? pkg.get("test").get("dependencies") : pkg.get("dependencies"); + const deps = isTest + ? pkg.get("test").get("dependencies") + : pkg.get("dependencies"); let newItems = []; for (const el of deps.items) { if ( @@ -124,7 +130,16 @@ export function migrateV1ConfigImpl(doc) { return match.charAt(1).toUpperCase(); }); } - } + + if (pair.key.value === "censorTestWarnings") { + hasChanged = true; + const root = _path.at(0); + const build = root.get("package").get("build"); + build.set("censorTestWarnings", pair.value); + + return Yaml.visit.REMOVE; + } + }, }); if (hasChanged) { return doc; diff --git a/src/Spago/Psa.purs b/src/Spago/Psa.purs index b51643c1d..f4e7faa1a 100644 --- a/src/Spago/Psa.purs +++ b/src/Spago/Psa.purs @@ -160,7 +160,7 @@ toWorkspacePackagePathDecision { selected: { path, package }, psaCliFlags } = do { pathIsFromPackage: (testPath `Path.isPrefixOf` _) , pathType: IsSrc , strict: fromMaybe false $ psaCliFlags.strict <|> (package.test >>= _.strict) - , censorWarnings: package.test >>= _.censorTestWarnings + , censorWarnings: package.build >>= _.censorTestWarnings } ] diff --git a/test-fixtures/build/migrate-config/migrated-spago.yaml b/test-fixtures/build/migrate-config/migrated-spago.yaml index 07bf8faab..ecdaa5898 100644 --- a/test-fixtures/build/migrate-config/migrated-spago.yaml +++ b/test-fixtures/build/migrate-config/migrated-spago.yaml @@ -10,10 +10,10 @@ package: main: Test.Main execArgs: [] dependencies: [] - censorTestWarnings: all pedanticPackages: false build: censorProjectWarnings: all + censorTestWarnings: all pedanticPackages: false workspace: packageSet: diff --git a/test/Prelude.purs b/test/Prelude.purs index 1048b52a4..3e9562b80 100644 --- a/test/Prelude.purs +++ b/test/Prelude.purs @@ -105,20 +105,20 @@ shouldEqualStr v1 v2 = let renderNonPrinting = String.replaceAll (String.Pattern "\r") (String.Replacement "␍") - >>> String.replaceAll (String.Pattern "\t") (String.Replacement "␉-->") + >>> String.replaceAll (String.Pattern "\t") (String.Replacement "␉-->") in - when (v1 /= v2) do - fail $ Array.intercalate "\n" - [ "" - , "===== (Actual)" - , renderNonPrinting v1 - , "=====" - , " ≠" - , "===== (Expected)" - , renderNonPrinting v2 - , "=====" - , "" - ] + when (v1 /= v2) do + fail $ Array.intercalate "\n" + [ "" + , "===== (Actual)" + , renderNonPrinting v1 + , "=====" + , " ≠" + , "===== (Expected)" + , renderNonPrinting v2 + , "=====" + , "" + ] checkFixture :: ∀ path. IsPath path => path -> FixturePath -> Aff Unit checkFixture filepath fixturePath = checkFixture' filepath fixturePath (shouldEqualStr `on` String.trim) @@ -324,83 +324,63 @@ configurePackageSection initialOptions = snd <<< Array.foldl (\c f -> f c) configAddSrcStrict :: Tuple String Init.DefaultConfigPackageOptions -> Tuple String Init.DefaultConfigPackageOptions configAddSrcStrict = map \r -> r - { build = Just - { strict: Just true - , censorProjectWarnings: r.build >>= _.censorProjectWarnings - , pedanticPackages: r.build >>= _.pedanticPackages + { build = r.build <#> _ + { strict = Just true } } configAddSrcPedantic :: Tuple String Init.DefaultConfigPackageOptions -> Tuple String Init.DefaultConfigPackageOptions configAddSrcPedantic = map \r -> r - { build = Just - { strict: r.build >>= _.strict - , censorProjectWarnings: r.build >>= _.censorProjectWarnings - , pedanticPackages: Just true + { build = r.build <#> _ + { pedanticPackages = Just true } } configAddSrcCensor :: Config.CensorBuildWarnings -> Tuple String Init.DefaultConfigPackageOptions -> Tuple String Init.DefaultConfigPackageOptions configAddSrcCensor censors = map \r -> r - { build = Just - { strict: r.build >>= _.strict - , censorProjectWarnings: Just censors - , pedanticPackages: r.build >>= _.pedanticPackages + { build = r.build <#> _ + { censorProjectWarnings = Just censors } } configAddTestMain :: Tuple String Init.DefaultConfigPackageOptions -> Tuple String Init.DefaultConfigPackageOptions configAddTestMain (Tuple packageName r) = Tuple packageName $ r - { test = Just - { moduleMain: mkTestModuleName packageName - , strict: r.test >>= _.strict - , censorTestWarnings: r.test >>= _.censorTestWarnings - , pedanticPackages: r.test >>= _.pedanticPackages - , dependencies: r.test >>= _.dependencies + { test = r.test <#> _ + { moduleMain = mkTestModuleName packageName } } configAddTestStrict :: Tuple String Init.DefaultConfigPackageOptions -> Tuple String Init.DefaultConfigPackageOptions configAddTestStrict (Tuple packageName r) = Tuple packageName $ r - { test = Just - { moduleMain: mkTestModuleName packageName - , strict: Just true - , censorTestWarnings: r.test >>= _.censorTestWarnings - , pedanticPackages: r.test >>= _.pedanticPackages - , dependencies: r.test >>= _.dependencies + { test = r.test <#> _ + { moduleMain = mkTestModuleName packageName + , strict = Just true } } configAddTestPedantic :: Tuple String Init.DefaultConfigPackageOptions -> Tuple String Init.DefaultConfigPackageOptions configAddTestPedantic (Tuple packageName r) = Tuple packageName $ r - { test = Just - { moduleMain: mkTestModuleName packageName - , strict: r.test >>= _.strict - , censorTestWarnings: r.test >>= _.censorTestWarnings - , pedanticPackages: Just true - , dependencies: r.test >>= _.dependencies + { test = r.test <#> _ + { moduleMain = mkTestModuleName packageName + , pedanticPackages = Just true } } configAddTestCensor :: Config.CensorBuildWarnings -> Tuple String Init.DefaultConfigPackageOptions -> Tuple String Init.DefaultConfigPackageOptions configAddTestCensor censors (Tuple packageName r) = Tuple packageName $ r - { test = Just - { moduleMain: mkTestModuleName packageName - , strict: r.test >>= _.strict - , censorTestWarnings: Just censors - , pedanticPackages: r.test >>= _.pedanticPackages - , dependencies: r.test >>= _.dependencies + { test = r.test <#> _ + { moduleMain = mkTestModuleName packageName + } + , build = r.build <#> _ + { censorTestWarnings = Just censors } } configAddTestDependencies :: Array String -> Tuple String Init.DefaultConfigPackageOptions -> Tuple String Init.DefaultConfigPackageOptions configAddTestDependencies deps (Tuple packageName r) = Tuple packageName $ r - { test = Just - { moduleMain: mkTestModuleName packageName - , strict: r.test >>= _.strict - , censorTestWarnings: r.test >>= _.censorTestWarnings - , pedanticPackages: r.test >>= _.pedanticPackages - , dependencies: Just $ maybe (mkDependencies deps) (append (mkDependencies deps)) $ r.test >>= _.dependencies + { test = r.test <#> _ + { moduleMain = mkTestModuleName packageName + , dependencies = Just $ maybe (mkDependencies deps) (append (mkDependencies deps)) $ r.test >>= _.dependencies } } diff --git a/test/Spago/Build.purs b/test/Spago/Build.purs index cc780fe87..bfcaa79bf 100644 --- a/test/Spago/Build.purs +++ b/test/Spago/Build.purs @@ -39,8 +39,8 @@ spec = Spec.around withTempDir do , result: isLeft , sanitize: String.trim - >>> String.replaceAll (String.Pattern "Usage: purs.bin") (String.Replacement "Usage: purs") - >>> String.replaceAll (String.Pattern "\r\n") (String.Replacement "\n") + >>> String.replaceAll (String.Pattern "Usage: purs.bin") (String.Replacement "Usage: purs") + >>> String.replaceAll (String.Pattern "\r\n") (String.Replacement "\n") } Spec.it "passes options to purs" \{ spago } -> do @@ -220,7 +220,7 @@ spec = Spec.around withTempDir do Just Platform.Win32 -> fixture "build/json-truncated-many-warnings/warnings-windows.json" _ -> fixture "build/json-truncated-many-warnings/warnings.json" - Spec.it "building with old-format config files works, as well as migrating them" \{ spago, fixture, testCwd } -> do + Spec.itOnly "building with old-format config files works, as well as migrating them" \{ spago, fixture, testCwd } -> do FS.copyTree { src: fixture "build/migrate-config", dst: testCwd "." } spago [ "build" ] >>= shouldBeSuccess spago [ "build" ] >>= shouldBeSuccessErr (fixture "build/migrate-config/unmigrated-warning.txt") diff --git a/test/Spago/Build/Pedantic.purs b/test/Spago/Build/Pedantic.purs index 700952015..92ac96011 100644 --- a/test/Spago/Build/Pedantic.purs +++ b/test/Spago/Build/Pedantic.purs @@ -201,10 +201,11 @@ spec = spago [ "install", "-p", "follow-instructions", "effect" ] >>= shouldBeSuccessErr (fixture "pedantic/pedantic-instructions-installation-result.txt") -- Regression test for https://github.com/purescript/spago/pull/1222 - let gitignores = [".spago", "/.spago", ".spago/**"] + let gitignores = [ ".spago", "/.spago", ".spago/**" ] for_ gitignores \gitignore -> Spec.it - (".gitignore does not affect discovery of transitive deps (" <> gitignore <> ")") \{ spago, fixture, testCwd } -> do + (".gitignore does not affect discovery of transitive deps (" <> gitignore <> ")") + \{ spago, fixture, testCwd } -> do FS.copyTree { src: fixture "pedantic/follow-instructions", dst: testCwd } FS.writeTextFile (Path.global ".gitignore") gitignore spago [ "uninstall", "effect" ] >>= shouldBeSuccess @@ -221,6 +222,7 @@ addPedanticFlagToSrc config = config { pedanticPackages: Just true , strict: Nothing , censorProjectWarnings: Nothing + , censorTestWarnings: Nothing } } } @@ -232,7 +234,6 @@ addPedanticFlagToTest config = config { main: "Test.Main" , pedanticPackages: Just true , strict: Nothing - , censorTestWarnings: Nothing , dependencies: maybe (Dependencies Map.empty) _.dependencies r.test , execArgs: r.test >>= _.execArgs } diff --git a/test/Spago/Config.purs b/test/Spago/Config.purs index 89a0f138f..901365160 100644 --- a/test/Spago/Config.purs +++ b/test/Spago/Config.purs @@ -112,6 +112,7 @@ validSpagoYaml = , build: Just { strict: Just true , censorProjectWarnings: Nothing + , censorTestWarnings: Nothing , pedanticPackages: Nothing } , bundle: Nothing @@ -121,7 +122,6 @@ validSpagoYaml = , test: Just { main: "Test.Main" , dependencies: mkDependencies [ "spec", "spec-node" ] - , censorTestWarnings: Nothing , execArgs: Nothing , strict: Nothing , pedanticPackages: Nothing diff --git a/test/Spago/Test.purs b/test/Spago/Test.purs index f518cb367..e04258022 100644 --- a/test/Spago/Test.purs +++ b/test/Spago/Test.purs @@ -109,8 +109,8 @@ spec = Spec.around withTempDir do FS.writeYamlFile Config.configCodec (testCwd "spago.yaml") $ Init.defaultConfig' $ PackageAndWorkspace { name: mkPackageName "package-a" , dependencies: [ "prelude", "effect", "console" ] - , test: Just { moduleMain: "Test.Main", strict: Nothing, censorTestWarnings: Nothing, pedanticPackages: Nothing, dependencies: Nothing } - , build: Just { strict: Just true, censorProjectWarnings: Nothing, pedanticPackages: Nothing } + , test: Just { moduleMain: "Test.Main", strict: Nothing, pedanticPackages: Nothing, dependencies: Nothing } + , build: Just { strict: Just true, censorProjectWarnings: Nothing, censorTestWarnings: Nothing, pedanticPackages: Nothing } } { setVersion: Just $ unsafeFromRight $ Version.parse "0.0.1" } From a9958425c1012ec58ff7e0894c169839a39f3f7a Mon Sep 17 00:00:00 2001 From: Brad Reed Date: Thu, 6 Mar 2025 21:23:09 +0700 Subject: [PATCH 2/5] maybe fix test idk --- test/Spago/Build.purs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/Spago/Build.purs b/test/Spago/Build.purs index bfcaa79bf..8569f4f36 100644 --- a/test/Spago/Build.purs +++ b/test/Spago/Build.purs @@ -222,7 +222,7 @@ spec = Spec.around withTempDir do Spec.itOnly "building with old-format config files works, as well as migrating them" \{ spago, fixture, testCwd } -> do FS.copyTree { src: fixture "build/migrate-config", dst: testCwd "." } - spago [ "build" ] >>= shouldBeSuccess + -- spago [ "build" ] >>= shouldBeSuccess spago [ "build" ] >>= shouldBeSuccessErr (fixture "build/migrate-config/unmigrated-warning.txt") spago [ "build", "--migrate" ] >>= shouldBeSuccessErr (fixture "build/migrate-config/migrating-output.txt") spago [ "build" ] >>= shouldBeSuccessErr (fixture "build/migrate-config/migrated-output.txt") From a5bf7ea0544d7baaa4e42c979741c42927298f46 Mon Sep 17 00:00:00 2001 From: Brad Reed Date: Sun, 9 Mar 2025 19:26:50 +0400 Subject: [PATCH 3/5] fix test --- spago.yaml | 3 --- src/Spago/Config.js | 15 ++++++++++----- .../build/migrate-config/migrated-spago.yaml | 2 +- test/Spago/Build.purs | 4 ++-- 4 files changed, 13 insertions(+), 11 deletions(-) diff --git a/spago.yaml b/spago.yaml index 0dd5c7229..060fd8cfa 100644 --- a/spago.yaml +++ b/spago.yaml @@ -73,9 +73,6 @@ package: - unsafe-coerce test: main: Test.Spago - censorTestWarnings: - - ImplicitQualifiedImportReExport - - ImplicitQualifiedImport dependencies: - exceptions - quickcheck diff --git a/src/Spago/Config.js b/src/Spago/Config.js index 75dffa049..4d3c60f1a 100644 --- a/src/Spago/Config.js +++ b/src/Spago/Config.js @@ -131,16 +131,21 @@ export function migrateV1ConfigImpl(doc) { }); } + // move censorTestWarnings from test to build map if (pair.key.value === "censorTestWarnings") { - hasChanged = true; - const root = _path.at(0); - const build = root.get("package").get("build"); - build.set("censorTestWarnings", pair.value); + const parent = _path.at(-2); + if (parent.key && parent.key.value === "test") { + hasChanged = true; + const root = _path.at(0); + const build = root.get("package").get("build"); + build.set("censorTestWarnings", pair.value); - return Yaml.visit.REMOVE; + return Yaml.visit.REMOVE; + } } }, }); + if (hasChanged) { return doc; } diff --git a/test-fixtures/build/migrate-config/migrated-spago.yaml b/test-fixtures/build/migrate-config/migrated-spago.yaml index ecdaa5898..fdf3bf172 100644 --- a/test-fixtures/build/migrate-config/migrated-spago.yaml +++ b/test-fixtures/build/migrate-config/migrated-spago.yaml @@ -13,8 +13,8 @@ package: pedanticPackages: false build: censorProjectWarnings: all - censorTestWarnings: all pedanticPackages: false + censorTestWarnings: all workspace: packageSet: registry: 50.4.0 diff --git a/test/Spago/Build.purs b/test/Spago/Build.purs index 8569f4f36..f86d95eea 100644 --- a/test/Spago/Build.purs +++ b/test/Spago/Build.purs @@ -220,9 +220,9 @@ spec = Spec.around withTempDir do Just Platform.Win32 -> fixture "build/json-truncated-many-warnings/warnings-windows.json" _ -> fixture "build/json-truncated-many-warnings/warnings.json" - Spec.itOnly "building with old-format config files works, as well as migrating them" \{ spago, fixture, testCwd } -> do + Spec.it "building with old-format config files works, as well as migrating them" \{ spago, fixture, testCwd } -> do FS.copyTree { src: fixture "build/migrate-config", dst: testCwd "." } - -- spago [ "build" ] >>= shouldBeSuccess + spago [ "build" ] >>= shouldBeSuccess spago [ "build" ] >>= shouldBeSuccessErr (fixture "build/migrate-config/unmigrated-warning.txt") spago [ "build", "--migrate" ] >>= shouldBeSuccessErr (fixture "build/migrate-config/migrating-output.txt") spago [ "build" ] >>= shouldBeSuccessErr (fixture "build/migrate-config/migrated-output.txt") From ab7ece6ffedaf38641a17a7a6ee31a12377b666b Mon Sep 17 00:00:00 2001 From: Brad Reed Date: Mon, 10 Mar 2025 10:03:36 +0000 Subject: [PATCH 4/5] update changelog --- CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 189c003e2..bbb23bbce 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -48,6 +48,7 @@ Other improvements: - `spago publish` no longer tries to validate all workspace dependencies, but only the (transitive) dependencies of the project being published. - Restored broken search-directed search in generated docs. +- Move `censorTestWarnings` from `test` to `build` in `spago.yaml` ## [0.21.0] - 2023-05-04 From 35cf939cc25c01650d964b79e3fd833060a00679 Mon Sep 17 00:00:00 2001 From: Brad Reed Date: Mon, 10 Mar 2025 13:54:18 +0000 Subject: [PATCH 5/5] add back to spago.yaml --- spago.yaml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/spago.yaml b/spago.yaml index 060fd8cfa..cb2c85c74 100644 --- a/spago.yaml +++ b/spago.yaml @@ -11,6 +11,9 @@ package: censorProjectWarnings: - WildcardInferredType - ImplicitQualifiedImportReExport + censorTestWarnings: + - ImplicitQualifiedImportReExport + - ImplicitQualifiedImport dependencies: - aff - aff-promise