{"action":"closed","issue":{"url":"https://api.github.com/repos/bsansouci/bsb-native/issues/2","repository_url":"https://api.github.com/repos/bsansouci/bsb-native","labels_url":"https://api.github.com/repos/bsansouci/bsb-native/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/bsansouci/bsb-native/issues/2/comments","events_url":"https://api.github.com/repos/bsansouci/bsb-native/issues/2/events","html_url":"https://github.com/bsansouci/bsb-native/issues/2","id":240034270,"number":2,"title":"Some feedback","user":{"login":"saschatimme","id":4854317,"avatar_url":"https://avatars0.githubusercontent.com/u/4854317?v=4","gravatar_id":"","url":"https://api.github.com/users/saschatimme","html_url":"https://github.com/saschatimme","followers_url":"https://api.github.com/users/saschatimme/followers","following_url":"https://api.github.com/users/saschatimme/following{/other_user}","gists_url":"https://api.github.com/users/saschatimme/gists{/gist_id}","starred_url":"https://api.github.com/users/saschatimme/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/saschatimme/subscriptions","organizations_url":"https://api.github.com/users/saschatimme/orgs","repos_url":"https://api.github.com/users/saschatimme/repos","events_url":"https://api.github.com/users/saschatimme/events{/privacy}","received_events_url":"https://api.github.com/users/saschatimme/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2017-07-02T21:03:09Z","updated_at":"2017-11-14T06:34:39Z","closed_at":"2017-11-14T06:34:39Z","author_association":"CONTRIBUTOR","body":"I converted a mediumish (around 5k loc) to bsb-native yested and it was quite smooth! My 360 unit tests now need 1200 ms in js, 400ms with bytecode and 50ms with native 💯.\r\n\r\nHere are a couple of things I think could be improved:\r\n* The default flags to `bsc` seem different. Especially I got warning 40 which was quite annoying.  I added `[\"-no-alias-deps\", \"-color\", \"always\", \"-w\", \"-40+6+7+27+32..39+44+45\"]` and everything was fine again.\r\n*  It seems that only the first entry in `entries` will be build.\r\n* Is there a command line argument to `bsb` to specify which kind, .i.e. js, bytecode or native, will be build? Currently it seems that if there is an entry present in `entries` this will determine the kind. So if I want to build normally for js and only sometimes for native, I have to comment things in/out in the bsconfig.\r\n* Currently your platform specific dependencies have to be separate packages. Separate packages make it quite hard to iterate fast on things. To lessen the pain I ended up with some `npm link` magic to have everything still in the same repo. (I can elaborate if you are curious about the details). I think a better solution would be to have the possibility to set on a per directory basis which kinds are allowed.\r\nThis would also reduce the amount of restructuring necessary for a dual build and eliminate the need to extract some sort of interface (like in your ReasonGl thing).\r\n\r\n"}}
{"actor":{"display_login":"bsansouci"}}