You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Fix warnings in core libs revealed v0.14.1 PS release #4071
Now that v0.14. is out, we should fix any of the warnings found by the update in core libs. This issue tracks which of them need that and whether a PR has been submitted/merged yet. A checked box below means a PR has been submitted (but not necessarily merged). See methodology below for how this list was produced.
arrays
catenable-lists
datetime
filterable
foldable-traversable
foreign
free
functors
lists
ordered-collections
prelude
quickcheck
st
tailrec
transformers
tuples
Method for producing the above list:
Created a new project using spago init
Manually updated packages.dhall file to update metadata.version = "v0.14.1"
Ran spago install $(spago ls packages | cut -f 1 -d ' ' | tr '\n' ' ') to install all packages in the set
Ran spago build --purs-args "--censor-codes='UserDefinedWarning'". I was trying to stop MonadZero deprecation notices from being include din list of warnings. I don't think this worked because censor-codes likely only works on source code defined in the src/ directory, not the .spago/ directory.
Copied shell's output that listed all warnings into file called fix-warnings-source
Ran cat fix-warnings-source | grep '.spago/' | cut -f 3 -d ' ' | cut -f 2 -d '/' | uniq > fix-warnings.txt (record a list of deduplicated packages that have a warning in them)
Copied and pasted data into Excel file
Ran spago ls packages | grep '/purescript/' | cut -f 1 -d ' ' > purescript.txt to get a list of all core repos
Ran spago ls packages | grep '/purescript-contrib/' | cut -f 1 -d ' ' > purescript-contrib.txt to get a list of all contrib repos
Pasted purescript.txt and purescript-contib.txt into Excel fil
Used VLOOKUP Excel formula to find all purescript and purescript-contrib libs that had warnings.
This likely isn't the best method for determining things, but seems like it's likely 90% right.
Activity
JordanMartinez commentedon Apr 20, 2021
Looks like 5 left for core libs
thomashoneyman commentedon Apr 21, 2021
@JordanMartinez would it be possible to do the same for the
webandnodelibraries and add them to the contrib governance issue?JordanMartinez commentedon Apr 21, 2021
Done.
JordanMartinez commentedon Apr 25, 2021
Only 2 more PRs to merge and then this issue can be closed once new releases are made of the libraries.