{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/2611","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/2611/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/2611/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/2611/events","html_url":"https://github.com/RIOT-OS/RIOT/issues/2611","id":62082873,"number":2611,"title":"boards/Makefile.features: one feature per row","user":{"login":"haukepetersen","id":620834,"avatar_url":"https://avatars.githubusercontent.com/u/620834?v=3","gravatar_id":"","url":"https://api.github.com/users/haukepetersen","html_url":"https://github.com/haukepetersen","followers_url":"https://api.github.com/users/haukepetersen/followers","following_url":"https://api.github.com/users/haukepetersen/following{/other_user}","gists_url":"https://api.github.com/users/haukepetersen/gists{/gist_id}","starred_url":"https://api.github.com/users/haukepetersen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/haukepetersen/subscriptions","organizations_url":"https://api.github.com/users/haukepetersen/orgs","repos_url":"https://api.github.com/users/haukepetersen/repos","events_url":"https://api.github.com/users/haukepetersen/events{/privacy}","received_events_url":"https://api.github.com/users/haukepetersen/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/enhancement","name":"enhancement","color":"f17070"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Newbie-Task-Candidate","name":"Newbie-Task-Candidate","color":"9fe59f"}],"state":"open","locked":false,"assignee":{"login":"LudwigOrtmann","id":3739173,"avatar_url":"https://avatars.githubusercontent.com/u/3739173?v=3","gravatar_id":"","url":"https://api.github.com/users/LudwigOrtmann","html_url":"https://github.com/LudwigOrtmann","followers_url":"https://api.github.com/users/LudwigOrtmann/followers","following_url":"https://api.github.com/users/LudwigOrtmann/following{/other_user}","gists_url":"https://api.github.com/users/LudwigOrtmann/gists{/gist_id}","starred_url":"https://api.github.com/users/LudwigOrtmann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LudwigOrtmann/subscriptions","organizations_url":"https://api.github.com/users/LudwigOrtmann/orgs","repos_url":"https://api.github.com/users/LudwigOrtmann/repos","events_url":"https://api.github.com/users/LudwigOrtmann/events{/privacy}","received_events_url":"https://api.github.com/users/LudwigOrtmann/received_events","type":"User","site_admin":false},"milestone":null,"comments":5,"created_at":"2015-03-16T13:33:20Z","updated_at":"2015-03-22T18:28:26Z","closed_at":null,"body":"Most boards list their feature in one or two rows, e.g.:\r\n```\r\nFEATURES_PROVIDED += periph_uart periph_gpio periph_spi periph_i2c periph_pwm periph_random \\\r\n                     periph_adc periph_dac periph_cpuid\r\n```\r\nThis leads to conflicts when there are two PR's open simultaneously and both add something to this line. I would actually prefer to have one line per feature, e.g.:\r\n```\r\nFEATURES_PROVIDED += periph_uart\r\nFEATURES_PROVIDED += periph_spi\r\nFEATURES_PROVIDED += periph_i2c\r\nFEATURES_PROVIDED += periph_pwm\r\n...\r\n```\r\nThis way there won't be any merge conflicts anymore.\r\n\r\nHere is the TODO list:\r\n - [x] airfy-beacon, #2630\r\n - [x] arduino-due, #2630\r\n - [ ] ~~arduino-mega2560~~ no need, just one feature\r\n - [x] avsextrem, #2630\r\n - [x] cc2538dk, #2630\r\n - [x] chronos, #2630\r\n - [x] f4vi1\r\n - [x] fox, #2630\r\n - [x] iot-lab_M3, #2630\r\n - [x] mbed_lpc1768\r\n - [ ] ~~msb-430~~ no need, no features\r\n - [ ] ~~msb-430-common~~ no need, no features\r\n - [x] msb-430h, #2630\r\n - [x] msba2, #2630\r\n - [ ] ~~msba2-common~~ no need, no features\r\n - [x] msbiot, #2630\r\n - [x] mulle, #2630\r\n - [x] native, #2630\r\n - [x] nucleo-f091, #2630\r\n - [x] nucleo-f334\r\n - [x] nucleo-l1\r\n - [x] openmote, #2630\r\n - [x] pca10000, #2630\r\n - [x] pca10005, #2630\r\n - [x] pttu\r\n - [ ] ~~qemu-i386~~ no need, no features\r\n - [ ] ~~redbee-econotag~~ no need, just one feature\r\n - [x] samr21-xpro, #2630\r\n - [x] spark-core\r\n - [x] stm32f0discovery, #2630\r\n - [x] stm32f3discovery, #2630\r\n - [x] stm32f4discovery, #2630\r\n - [ ] ~~telosb~~ no need, just one feature\r\n - [x] udoo, #2630\r\n - [ ] ~~wsn430-common~~ no need, no features\r\n - [x] wsn430-v1_3b, #2630\r\n - [x] wsn430-v1_4, #2630\r\n - [ ] ~~x86-multiboot-common~~ no need, no features\r\n - [x] yunjia-nrf51822, #2630\r\n - [ ] ~~z1~~ no need, just one feature\r\n"},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/84670710","html_url":"https://github.com/RIOT-OS/RIOT/issues/2611#issuecomment-84670710","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/2611","id":84670710,"user":{"login":"Darredevil","id":6786777,"avatar_url":"https://avatars.githubusercontent.com/u/6786777?v=3","gravatar_id":"","url":"https://api.github.com/users/Darredevil","html_url":"https://github.com/Darredevil","followers_url":"https://api.github.com/users/Darredevil/followers","following_url":"https://api.github.com/users/Darredevil/following{/other_user}","gists_url":"https://api.github.com/users/Darredevil/gists{/gist_id}","starred_url":"https://api.github.com/users/Darredevil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Darredevil/subscriptions","organizations_url":"https://api.github.com/users/Darredevil/orgs","repos_url":"https://api.github.com/users/Darredevil/repos","events_url":"https://api.github.com/users/Darredevil/events{/privacy}","received_events_url":"https://api.github.com/users/Darredevil/received_events","type":"User","site_admin":false},"created_at":"2015-03-22T18:28:26Z","updated_at":"2015-03-22T18:28:26Z","body":"I wanted to tackle this but I noticed PR #2641 already does that but isn't merged,even Travis CI says everything is ok. Any reason for that?"}}