{"action":"created","issue":{"url":"https://api.github.com/repos/tomjaguarpaw/haskell-opaleye/issues/97","labels_url":"https://api.github.com/repos/tomjaguarpaw/haskell-opaleye/issues/97/labels{/name}","comments_url":"https://api.github.com/repos/tomjaguarpaw/haskell-opaleye/issues/97/comments","events_url":"https://api.github.com/repos/tomjaguarpaw/haskell-opaleye/issues/97/events","html_url":"https://github.com/tomjaguarpaw/haskell-opaleye/issues/97","id":100968555,"number":97,"title":"Need to change Column (Nullable ...) to NullableCollumn ...","user":{"login":"tomjaguarpaw","id":1951567,"avatar_url":"https://avatars.githubusercontent.com/u/1951567?v=3","gravatar_id":"","url":"https://api.github.com/users/tomjaguarpaw","html_url":"https://github.com/tomjaguarpaw","followers_url":"https://api.github.com/users/tomjaguarpaw/followers","following_url":"https://api.github.com/users/tomjaguarpaw/following{/other_user}","gists_url":"https://api.github.com/users/tomjaguarpaw/gists{/gist_id}","starred_url":"https://api.github.com/users/tomjaguarpaw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomjaguarpaw/subscriptions","organizations_url":"https://api.github.com/users/tomjaguarpaw/orgs","repos_url":"https://api.github.com/users/tomjaguarpaw/repos","events_url":"https://api.github.com/users/tomjaguarpaw/events{/privacy}","received_events_url":"https://api.github.com/users/tomjaguarpaw/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-08-14T09:13:26Z","updated_at":"2015-08-14T11:08:06Z","closed_at":null,"body":"I realised that `Column (Nullable ...)` doesn't make sense.  Not only does it allow the nonsensical type `Column (Nullable (Nullable ...))` it also means that the type `Column a -> Column a -> Column PGBool` is uncorrect.  It should be `Column a -> Column a -> Column (Nullable PGBool)`.  This is too painful to contemplate!\r\n\r\nThe correct fix is to have two type constructors, `Column` and `NullableColumn`.  This will break a lot of code.  I propose the following\r\n\r\n* 0.5: Add `NullableColumn a` as type alias for `Column (Nullable a)`\r\n* 0.6: Deprecate `Nullable`\r\n* 0.7: Remove `Nullable`\r\n\r\nAs the major stakeholders, I would appreciate opinions from\r\n\r\n* @bergmark \r\n* @ocharles \r\n* @hesselink \r\n* @dbp\r\n* @c-lewis \r\n* @coubeatczech \r\n* @tstat \r\n"},"comment":{"url":"https://api.github.com/repos/tomjaguarpaw/haskell-opaleye/issues/comments/131079009","html_url":"https://github.com/tomjaguarpaw/haskell-opaleye/issues/97#issuecomment-131079009","issue_url":"https://api.github.com/repos/tomjaguarpaw/haskell-opaleye/issues/97","id":131079009,"user":{"login":"tomjaguarpaw","id":1951567,"avatar_url":"https://avatars.githubusercontent.com/u/1951567?v=3","gravatar_id":"","url":"https://api.github.com/users/tomjaguarpaw","html_url":"https://github.com/tomjaguarpaw","followers_url":"https://api.github.com/users/tomjaguarpaw/followers","following_url":"https://api.github.com/users/tomjaguarpaw/following{/other_user}","gists_url":"https://api.github.com/users/tomjaguarpaw/gists{/gist_id}","starred_url":"https://api.github.com/users/tomjaguarpaw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomjaguarpaw/subscriptions","organizations_url":"https://api.github.com/users/tomjaguarpaw/orgs","repos_url":"https://api.github.com/users/tomjaguarpaw/repos","events_url":"https://api.github.com/users/tomjaguarpaw/events{/privacy}","received_events_url":"https://api.github.com/users/tomjaguarpaw/received_events","type":"User","site_admin":false},"created_at":"2015-08-14T11:08:06Z","updated_at":"2015-08-14T11:08:06Z","body":"> I think this would be pretty hairy for us to work around...\r\n\r\nCan you say more about why?\r\n\r\n> We solved this by not allowing comparisons of `Nullable`\r\n\r\nHow did you acheive that?"}}