Extra space added in forall #264

New Issue

2019-11-13T16:11:56+01:00

pbrisbin commented

2019-11-13 16:11:56 +01:00

(Migrated from github.com)

This diff is between code formatted by v0.12.0.0 and v0.12.1.0:

 -- | Construction of an action from some JSON-encodable
 someBatchUpdateAction
-  :: forall a . (Typeable a, ToJSON a) => a -> SomeBatchUpdateAction
+  :: forall  a . (Typeable a, ToJSON a) => a -> SomeBatchUpdateAction
 someBatchUpdateAction a =
   SomeBatchUpdateAction $ object ["tag" .= show (typeOf a), "contents" .= a]

I believe v0.12.0.0 is correct and there's a bug adding the extra space in v0.12.1.0.

This diff is between code formatted by v0.12.0.0 and v0.12.1.0: ```diff -- | Construction of an action from some JSON-encodable someBatchUpdateAction - :: forall a . (Typeable a, ToJSON a) => a -> SomeBatchUpdateAction + :: forall a . (Typeable a, ToJSON a) => a -> SomeBatchUpdateAction someBatchUpdateAction a = SomeBatchUpdateAction $ object ["tag" .= show (typeOf a), "contents" .= a] ``` I believe v0.12.0.0 is correct and there's a bug adding the extra space in v0.12.1.0.

lspitzner commented

2019-11-13 20:14:02 +01:00

(Migrated from github.com)

Thanks, this is fixed on master, see 974826f98f.

Thanks, this is fixed on master, see 974826f98fa9f08e659dc334edda61bb5c7cfa49.

lspitzner commented

2019-12-09 22:59:48 +01:00

(Migrated from github.com)

fixed in 0.12.1.1

Sign in to join this conversation.

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: hexagoxel/brittany#264