{"action":"created","issue":{"url":"https://api.github.com/repos/OPM/opm-simulators/issues/995","repository_url":"https://api.github.com/repos/OPM/opm-simulators","labels_url":"https://api.github.com/repos/OPM/opm-simulators/issues/995/labels{/name}","comments_url":"https://api.github.com/repos/OPM/opm-simulators/issues/995/comments","events_url":"https://api.github.com/repos/OPM/opm-simulators/issues/995/events","html_url":"https://github.com/OPM/opm-simulators/pull/995","id":198020503,"number":995,"title":"BlackoilModelEnums: don't use high-level classes","user":{"login":"andlaus","id":1837735,"avatar_url":"https://avatars.githubusercontent.com/u/1837735?v=3","gravatar_id":"","url":"https://api.github.com/users/andlaus","html_url":"https://github.com/andlaus","followers_url":"https://api.github.com/users/andlaus/followers","following_url":"https://api.github.com/users/andlaus/following{/other_user}","gists_url":"https://api.github.com/users/andlaus/gists{/gist_id}","starred_url":"https://api.github.com/users/andlaus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andlaus/subscriptions","organizations_url":"https://api.github.com/users/andlaus/orgs","repos_url":"https://api.github.com/users/andlaus/repos","events_url":"https://api.github.com/users/andlaus/events{/privacy}","received_events_url":"https://api.github.com/users/andlaus/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-12-29T15:02:49Z","updated_at":"2017-01-01T19:41:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/OPM/opm-simulators/pulls/995","html_url":"https://github.com/OPM/opm-simulators/pull/995","diff_url":"https://github.com/OPM/opm-simulators/pull/995.diff","patch_url":"https://github.com/OPM/opm-simulators/pull/995.patch"},"body":"I consider the blackoil model enums to be pretty low level while the \"FluidProperties\" code is IMO quite high level. this, using the latter to define the former constitutes a layering violation IMO. note that the fix is to simply use the enums from opm-core directly."},"comment":{"url":"https://api.github.com/repos/OPM/opm-simulators/issues/comments/269916375","html_url":"https://github.com/OPM/opm-simulators/pull/995#issuecomment-269916375","issue_url":"https://api.github.com/repos/OPM/opm-simulators/issues/995","id":269916375,"user":{"login":"atgeirr","id":1876621,"avatar_url":"https://avatars.githubusercontent.com/u/1876621?v=3","gravatar_id":"","url":"https://api.github.com/users/atgeirr","html_url":"https://github.com/atgeirr","followers_url":"https://api.github.com/users/atgeirr/followers","following_url":"https://api.github.com/users/atgeirr/following{/other_user}","gists_url":"https://api.github.com/users/atgeirr/gists{/gist_id}","starred_url":"https://api.github.com/users/atgeirr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/atgeirr/subscriptions","organizations_url":"https://api.github.com/users/atgeirr/orgs","repos_url":"https://api.github.com/users/atgeirr/repos","events_url":"https://api.github.com/users/atgeirr/events{/privacy}","received_events_url":"https://api.github.com/users/atgeirr/received_events","type":"User","site_admin":false},"created_at":"2017-01-01T19:41:44Z","updated_at":"2017-01-01T19:41:44Z","body":"With the last commit, there should be no risk of inconsistency, so I'll merge. Thanks!"}}
{"actor":{"display_login":"atgeirr"}}