purescript-contrib/purescript-formatters#76

Should I comment out formatters from the package-set and then we add formatters back in when it upgrades?

Yes, we typically decide whether to delay upgrading the root package to avoid breaking things downstream or whether to drop things downstream until they can be updated. In this case would you mind if we get the formatters PR merged and then update both in package sets at once?

get the formatters PR merged and then update both in package sets at once?

Sounds good.

@jamesdbrock https://github.com/purescript-contrib/purescript-formatters/releases/tag/v6.0.0

Looks like this is at least affecting graphql-client as well:
https://github.com/OxfordAbstracts/purescript-graphql-client

Error found:
in module Data.GraphQL.Parser
at .spago/graphql-client/v4.0.18/src/Data/GraphQL/Parser.purs:21:36 - 21:52 (line 21, column 36 - line 21, column 52)

  Cannot import type class StringLike from module Text.Parsing.Parser.String
  It either does not exist or the module does not export it.


See https://github.com/purescript/documentation/blob/master/errors/UnknownImport.md for more information,
or to contribute content related to this error.

Looks like this is at least affecting graphql-client as well: https://github.com/OxfordAbstracts/purescript-graphql-client

Yeah, what do you think I should do here @thomashoneyman ?

1. Comment out graphql-client from the package sets in this PR?
2. Upgrade graphql-client first and then add it to this PR?

New issue: OxfordAbstracts/purescript-graphql-client#65

@jamesdbrock What do you think of leaving that issue open for a week, and if there's no response in that time we can comment out the library and release a new package set, adding graphql-client back in when it's ready?

For now we could go ahead and comment it out in case there are other failures down the line.

Thanks for the heads up. I'll remove StringLike today

https://github.com/OxfordAbstracts/purescript-graphql-client/releases/tag/v7.0.0

@jamesdbrock I think it should be graphql-client v7.0.0

Ok I think this is ready to go now. I'll open issues with @markfarrell when it's merged so he can upgrade his packages.

I've merged this, and yea we should open issues (or even pull requests if it's simple) against the two disabled libraries. Thanks @jamesdbrock!