{"action":"created","issue":{"url":"https://api.github.com/repos/meteor/guide/issues/226","repository_url":"https://api.github.com/repos/meteor/guide","labels_url":"https://api.github.com/repos/meteor/guide/issues/226/labels{/name}","comments_url":"https://api.github.com/repos/meteor/guide/issues/226/comments","events_url":"https://api.github.com/repos/meteor/guide/issues/226/events","html_url":"https://github.com/meteor/guide/pull/226","id":132065553,"number":226,"title":"Structure content","user":{"login":"tmeasday","id":132554,"avatar_url":"https://avatars.githubusercontent.com/u/132554?v=3","gravatar_id":"","url":"https://api.github.com/users/tmeasday","html_url":"https://github.com/tmeasday","followers_url":"https://api.github.com/users/tmeasday/followers","following_url":"https://api.github.com/users/tmeasday/following{/other_user}","gists_url":"https://api.github.com/users/tmeasday/gists{/gist_id}","starred_url":"https://api.github.com/users/tmeasday/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmeasday/subscriptions","organizations_url":"https://api.github.com/users/tmeasday/orgs","repos_url":"https://api.github.com/users/tmeasday/repos","events_url":"https://api.github.com/users/tmeasday/events{/privacy}","received_events_url":"https://api.github.com/users/tmeasday/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"stubailo","id":448783,"avatar_url":"https://avatars.githubusercontent.com/u/448783?v=3","gravatar_id":"","url":"https://api.github.com/users/stubailo","html_url":"https://github.com/stubailo","followers_url":"https://api.github.com/users/stubailo/followers","following_url":"https://api.github.com/users/stubailo/following{/other_user}","gists_url":"https://api.github.com/users/stubailo/gists{/gist_id}","starred_url":"https://api.github.com/users/stubailo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stubailo/subscriptions","organizations_url":"https://api.github.com/users/stubailo/orgs","repos_url":"https://api.github.com/users/stubailo/repos","events_url":"https://api.github.com/users/stubailo/events{/privacy}","received_events_url":"https://api.github.com/users/stubailo/received_events","type":"User","site_admin":false},"milestone":null,"comments":10,"created_at":"2016-02-08T06:37:30Z","updated_at":"2016-03-01T01:29:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/meteor/guide/pulls/226","html_url":"https://github.com/meteor/guide/pull/226","diff_url":"https://github.com/meteor/guide/pull/226.diff","patch_url":"https://github.com/meteor/guide/pull/226.patch"},"body":"This is a bit loose in parts and has a few XXXs in there but I think ti might be a good time to start discussing.\n<a href='#crh-start'></a><a href='#crh-data-%7B%22processed%22%3A%20%5B%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136629%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136685%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136724%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136764%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136792%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136820%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136880%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136886%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136908%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136914%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136933%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23issuecomment-181228413%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52138211%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52268913%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52269424%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52269675%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52269835%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23issuecomment-181721492%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23issuecomment-181739143%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23issuecomment-182130058%22%2C%20%22https%3A//github.com/meteor/guide/pull/226%23issuecomment-182133395%22%5D%2C%20%22comments%22%3A%20%7B%22General%20Comment%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/meteor/guide/pull/226%23issuecomment-181228413%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Note%20that%20this%20should%20be%20a%20complete%20replacement%20for%20the%20following%20section%20of%20the%20docs%3A%5Cr%5Cn%5Cr%5Cnhttp%3A//docs.meteor.com/%23/full/namespacing%5Cr%5Cn%5Cr%5CnWe%20should%20go%20through%20it%20before%20merging%20and%20make%20sure%20it%20has%20everything.%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A52%3A46Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%2C%20%7B%22body%22%3A%20%22%40stubailo%20I%20just%20renamed%20a%20%2Abunch%2A%20of%20files%20in%20https%3A//github.com/meteor/todos/commit/bcd27ff62a231249f9aabf572484bf7989188c40%5Cr%5Cn%5Cr%5CnCan%20you%20take%20a%20look%20and%20make%20sure%20it%20fits%20with%20what%20you%20expect%3F%22%2C%20%22created_at%22%3A%20%222016-02-09T05%3A54%3A11Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/132554%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/tmeasday%22%7D%7D%2C%20%7B%22body%22%3A%20%22%40tmeasday%20is%20initializers%20better%20than%20%60startup%60%3F%20Looks%20really%20great%21%22%2C%20%22created_at%22%3A%20%222016-02-09T07%3A02%3A51Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%2C%20%7B%22body%22%3A%20%22%60startup%60%20is%20shorter%20and%20easier%20to%20spell%20for%20-z-%20challenged%20folks%20like%20myself%2C%20and%20fits%20better%20with%20the%20existing%20%60Meteor.startup%28%29%60.%20Do%20you%20think%20it%20was%20the%20right%20call%20to%20have%20%60startup/client%60%20and%20%60startup/server%60%2C%20or%20is%20it%20better%20for%20the%20place%20it%27s%20included%20from%20determine%20the%20place%20where%20it%20runs%3F%22%2C%20%22created_at%22%3A%20%222016-02-09T23%3A20%3A37Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/132554%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/tmeasday%22%7D%7D%2C%20%7B%22body%22%3A%20%22%3E%20Do%20you%20think%20it%20was%20the%20right%20call%20to%20have%20startup/client%20and%20startup/server%2C%20or%20is%20it%20better%20for%20the%20place%20it%27s%20included%20from%20determine%20the%20place%20where%20it%20runs%3F%5Cr%5Cn%5Cr%5CnNo%20idea%2C%20I%20think%20this%20is%20a%20fairly%20arbitrary%20point.%20But%20I%20do%20think%20most%20initializers%20are%20written%20for%20one%20or%20the%20other%2C%20so%20it%20probably%20makes%20sense%20to%20have%20it%20the%20way%20you%20do%20now.%22%2C%20%22created_at%22%3A%20%222016-02-09T23%3A32%3A47Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%5D%2C%20%22title%22%3A%20%22General%20Comment%22%7D%2C%20%22Pull%20b36f93361d2d1521448ac0eff831477351637e38%20content/structure.md%20349%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136880%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22This%20won%27t%20actually%20work%20-%20for%20the%20below%20import%20to%20work%20this%20would%20have%20to%20be%20an%20%60export%20default%60.%5Cr%5Cn%5Cr%5CnPersonally%2C%20I%20don%27t%20know%20if%20it%27s%20better%20to%20have%20a%20default%20export%20or%20not%2C%20what%20do%20you%20think%20we%20should%20recommend%20here%20%40benjamn%3F%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A47%3A53Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20content/structure.md%3AL1-391%22%7D%2C%20%22Pull%20b36f93361d2d1521448ac0eff831477351637e38%20content/structure.md%2015%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136629%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22%5C%22ES2015%20packages%5C%22%20-%3E%20%5C%22modules%5C%22%5Cr%5Cn%5Cr%5CnIn%20general%2C%20this%20paragraph%20conflates%20modules%20and%20NPM%20packages%2C%20which%20are%20pretty%20different%20things.%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A40%3A26Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%2C%20%7B%22body%22%3A%20%22Also%2C%20I%20think%20we%20should%20greatly%20expand%20this%20section%20to%20be%20an%20introduction%20to%20modules%20in%20general%2C%20for%20people%20who%20aren%27t%20aware%20that%20a%20module%20is%20really%20a%20file%2C%20that%20each%20file%20can%20export%20one%20or%20more%20items%2C%20that%20some%20modules%20can%20be%20shared%20between%20client%20and%20server%2C%20etc.%5Cr%5Cn%5Cr%5CnWe%20don%27t%20have%20anywhere%20else%20that%20explains%20this%20stuff%2C%20so%20this%20is%20the%20place.%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A48%3A56Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%2C%20%7B%22body%22%3A%20%22Agreed%2C%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A49%3A35Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/132554%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/tmeasday%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20content/structure.md%3AL1-391%22%7D%2C%20%22Pull%20b36f93361d2d1521448ac0eff831477351637e38%20content/structure.md%2051%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136724%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Is%20%60fixtures%60%20a%20good%20name%20for%20this%3F%20I%20feel%20like%20that%20word%20is%20often%20associated%20with%20testing.%20In%20Rails%2C%20I%20think%20these%20are%20called%20%60seeds%60%3F%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A43%3A22Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%2C%20%7B%22body%22%3A%20%22Seeds%20are%20typically%20a%20set%20of%20info%20that%20any%20application%20needs%20to%20run%20%28like%20the%20list%20of%20countries%20in%20your%20signup%20form%29%2C%20where%20as%20fixtures%20are%20like%20a%20consistent%20%5C%22fake%5C%22%20set%20of%20data%20that%20you%20can%20use%20for%20various%20kinds%20of%20testing%20%28including%20playing%20with%20the%20app%20in%20development%21%29.%20So%20actually%20I%20think%20fixtures%20is%20the%20correct%20name%20here.%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A48%3A00Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/132554%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/tmeasday%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20content/structure.md%3AL1-391%22%7D%2C%20%22Pull%20b36f93361d2d1521448ac0eff831477351637e38%20content/structure.md%2025%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136685%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Is%20%60routes.js%60%20a%20good%20name%20for%20this%20file%3F%20Do%20we%20think%20in%20general%20that%20routes%20should%20be%20the%20entry%20point%20to%20the%20whole%20app%3F%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A41%3A53Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%2C%20%7B%22body%22%3A%20%22I%27m%20not%20sure%20really.%20What%20could%20also%20make%20sense%20to%20me%20would%20be%20to%20have%20%60client/main.js%60%20that%20imports%20the%20routes%2C%20because%3A%5Cr%5Cn%5Cr%5Cn%201.%20You%20might%20want%20to%20import%20the%20routes%20on%20the%20server%5Cr%5Cn%202.%20Your%20router%20file%20could%20be%20really%20huge%20and%20want%20to%20be%20split%20over%20multiple%20files.%5Cr%5Cn%5Cr%5CnAlso%20there%20can%20be%20other%20client%20init%20stuff%20like%3A%5Cr%5Cn%201.%20Configuring%20things%20like%20intercom%20/%20GA%20/%20segment%20/%20etc.%5Cr%5Cn%202.%20Configuring%20accounts%20related%20stuff.%5Cr%5Cn%5Cr%5CnI%27m%20not%20convinced%20it%27s%20not%20better%20to%20just%20have%202%20simple%20entrypoints%20%60client/main.js%60%20and%20%60server/main.js%60.%20Although%20this%20means%20you%20probably%20need%20like%20%60import/initializers%60%20for%20random%20config%20type%20stuff.%20I%27d%20be%20OK%20with%20that%20though.%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A46%3A01Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/132554%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/tmeasday%22%7D%7D%2C%20%7B%22body%22%3A%20%22Yeah%20I%20think%20your%20idea%20is%20better.%20Otherwise%20people%20need%20to%20start%20deciding%20what%20should%20be%20an%20entry%20point%2C%20and%20what%20should%20be%20a%20module.%22%2C%20%22created_at%22%3A%20%222016-02-08T07%3A23%3A35Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20content/structure.md%3AL1-391%22%7D%2C%20%22Pull%20b36f93361d2d1521448ac0eff831477351637e38%20content/structure.md%2083%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136792%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Should%20this%20be%20%60.tests.js%60%3F%20Also%2C%20does%20only%20one%20of%20these%20extensions%20work%2C%20or%20both%3F%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A45%3A26Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%2C%20%7B%22body%22%3A%20%22I%20think%20you%20are%20right.%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A48%3A44Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/132554%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/tmeasday%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20content/structure.md%3AL1-391%22%7D%2C%20%22Pull%20b36f93361d2d1521448ac0eff831477351637e38%20content/structure.md%2061%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/meteor/guide/pull/226%23discussion_r52136764%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22In%20today%27s%20unfortunate%20world%2C%20it%20is%20faster%20to%20recompile%20and%20restart%20your%20app%20if%20you%20edit%20client-only%20code.%20I%20found%20that%20putting%20my%20whole%20UI%20in%20common%20code%20made%20rebuilds%20unbearably%20slow%2C%20but%20maybe%20this%20is%20different%20after%20the%20perf%20work.%5Cr%5Cn%5Cr%5CnAlso%2C%20SSR%20is%20overrated.%20But%20that%27s%20a%20different%20question.%22%2C%20%22created_at%22%3A%20%222016-02-08T06%3A44%3A34Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%2C%20%7B%22body%22%3A%20%22Let%20me%20try%20that%20out%20now%22%2C%20%22created_at%22%3A%20%222016-02-09T05%3A24%3A02Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/132554%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/tmeasday%22%7D%7D%2C%20%7B%22body%22%3A%20%22Ok%20I%20turned%20on%20%60METEOR_PROFILE%60%20and%20the%20rebuild%20time%20when%20editing%20a%20component%20file%20is%20%7E%202.5s%20on%20my%20machine%20in%20both%20cases.%20If%20anything%20it%20was%20a%20little%20faster%20after%20I%20renamed%20the%20directory%20to%20%60import/views%60.%20So%20let%27s%20say%20it%20doesn%27t%20matter%3F%5Cr%5Cn%5Cr%5CnSo%20what%20do%20we%20think%20the%20best%20directory%20name%20is%3F%22%2C%20%22created_at%22%3A%20%222016-02-09T05%3A36%3A10Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/132554%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/tmeasday%22%7D%7D%2C%20%7B%22body%22%3A%20%22Maybe%20just%20%60view%60%20or%20%60ui%60%3F%20We%20don%27t%20want%20to%20use%20%60components%60%20since%20that%20means%20something%20else.%22%2C%20%22created_at%22%3A%20%222016-02-09T05%3A42%3A53Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/448783%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/stubailo%22%7D%7D%2C%20%7B%22body%22%3A%20%22I%20like%20%60ui/%60%2C%20it%27s%20short.%22%2C%20%22created_at%22%3A%20%222016-02-09T05%3A46%3A27Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/132554%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/tmeasday%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20content/structure.md%3AL1-391%22%7D%7D%7D'></a>\n<a href='https://www.codereviewhub.com/'><img src='http://www.codereviewhub.com/site/github-bar.png' height=40></a>\n- [ ] <a href='#crh-comment-Pull b36f93361d2d1521448ac0eff831477351637e38 content/structure.md 15'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/meteor/guide/pull/226#discussion_r52136629'>File: content/structure.md:L1-391</a></b>\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> \"ES2015 packages\" -> \"modules\"\nIn general, this paragraph conflates modules and NPM packages, which are pretty different things.\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> Also, I think we should greatly expand this section to be an introduction to modules in general, for people who aren't aware that a module is really a file, that each file can export one or more items, that some modules can be shared between client and server, etc.\nWe don't have anywhere else that explains this stuff, so this is the place.\n- <a href='https://github.com/tmeasday'><img border=0 src='https://avatars.githubusercontent.com/u/132554?v=3' height=16 width=16'></a> Agreed,\n- [ ] <a href='#crh-comment-Pull b36f93361d2d1521448ac0eff831477351637e38 content/structure.md 25'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/meteor/guide/pull/226#discussion_r52136685'>File: content/structure.md:L1-391</a></b>\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> Is `routes.js` a good name for this file? Do we think in general that routes should be the entry point to the whole app?\n- <a href='https://github.com/tmeasday'><img border=0 src='https://avatars.githubusercontent.com/u/132554?v=3' height=16 width=16'></a> I'm not sure really. What could also make sense to me would be to have `client/main.js` that imports the routes, because:\n1. You might want to import the routes on the server\n2. Your router file could be really huge and want to be split over multiple files.\nAlso there can be other client init stuff like:\n1. Configuring things like intercom / GA / segment / etc.\n2. Configuring accounts related stuff.\nI'm not convinced it's not better to just have 2 simple entrypoints `client/main.js` and `server/main.js`. Although this means you probably need like `import/initializers` for random config type stuff. I'd be OK with that though.\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> Yeah I think your idea is better. Otherwise people need to start deciding what should be an entry point, and what should be a module.\n- [ ] <a href='#crh-comment-Pull b36f93361d2d1521448ac0eff831477351637e38 content/structure.md 51'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/meteor/guide/pull/226#discussion_r52136724'>File: content/structure.md:L1-391</a></b>\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> Is `fixtures` a good name for this? I feel like that word is often associated with testing. In Rails, I think these are called `seeds`?\n- <a href='https://github.com/tmeasday'><img border=0 src='https://avatars.githubusercontent.com/u/132554?v=3' height=16 width=16'></a> Seeds are typically a set of info that any application needs to run (like the list of countries in your signup form), where as fixtures are like a consistent \"fake\" set of data that you can use for various kinds of testing (including playing with the app in development!). So actually I think fixtures is the correct name here.\n- [ ] <a href='#crh-comment-Pull b36f93361d2d1521448ac0eff831477351637e38 content/structure.md 61'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/meteor/guide/pull/226#discussion_r52136764'>File: content/structure.md:L1-391</a></b>\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> In today's unfortunate world, it is faster to recompile and restart your app if you edit client-only code. I found that putting my whole UI in common code made rebuilds unbearably slow, but maybe this is different after the perf work.\nAlso, SSR is overrated. But that's a different question.\n- <a href='https://github.com/tmeasday'><img border=0 src='https://avatars.githubusercontent.com/u/132554?v=3' height=16 width=16'></a> Let me try that out now\n- <a href='https://github.com/tmeasday'><img border=0 src='https://avatars.githubusercontent.com/u/132554?v=3' height=16 width=16'></a> Ok I turned on `METEOR_PROFILE` and the rebuild time when editing a component file is ~ 2.5s on my machine in both cases. If anything it was a little faster after I renamed the directory to `import/views`. So let's say it doesn't matter?\nSo what do we think the best directory name is?\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> Maybe just `view` or `ui`? We don't want to use `components` since that means something else.\n- <a href='https://github.com/tmeasday'><img border=0 src='https://avatars.githubusercontent.com/u/132554?v=3' height=16 width=16'></a> I like `ui/`, it's short.\n- [ ] <a href='#crh-comment-Pull b36f93361d2d1521448ac0eff831477351637e38 content/structure.md 83'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/meteor/guide/pull/226#discussion_r52136792'>File: content/structure.md:L1-391</a></b>\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> Should this be `.tests.js`? Also, does only one of these extensions work, or both?\n- <a href='https://github.com/tmeasday'><img border=0 src='https://avatars.githubusercontent.com/u/132554?v=3' height=16 width=16'></a> I think you are right.\n- [ ] <a href='#crh-comment-Pull b36f93361d2d1521448ac0eff831477351637e38 content/structure.md 349'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/meteor/guide/pull/226#discussion_r52136880'>File: content/structure.md:L1-391</a></b>\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> This won't actually work - for the below import to work this would have to be an `export default`.\nPersonally, I don't know if it's better to have a default export or not, what do you think we should recommend here @benjamn?\n- [ ] <a href='#crh-comment-General Comment'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/meteor/guide/pull/226#issuecomment-181228413'>General Comment</a></b>\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> Note that this should be a complete replacement for the following section of the docs:\nhttp://docs.meteor.com/#/full/namespacing\nWe should go through it before merging and make sure it has everything.\n- <a href='https://github.com/tmeasday'><img border=0 src='https://avatars.githubusercontent.com/u/132554?v=3' height=16 width=16'></a> @stubailo I just renamed a *bunch* of files in https://github.com/meteor/todos/commit/bcd27ff62a231249f9aabf572484bf7989188c40\nCan you take a look and make sure it fits with what you expect?\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> @tmeasday is initializers better than `startup`? Looks really great!\n- <a href='https://github.com/tmeasday'><img border=0 src='https://avatars.githubusercontent.com/u/132554?v=3' height=16 width=16'></a> `startup` is shorter and easier to spell for -z- challenged folks like myself, and fits better with the existing `Meteor.startup()`. Do you think it was the right call to have `startup/client` and `startup/server`, or is it better for the place it's included from determine the place where it runs?\n- <a href='https://github.com/stubailo'><img border=0 src='https://avatars.githubusercontent.com/u/448783?v=3' height=16 width=16'></a> > Do you think it was the right call to have startup/client and startup/server, or is it better for the place it's included from determine the place where it runs?\nNo idea, I think this is a fairly arbitrary point. But I do think most initializers are written for one or the other, so it probably makes sense to have it the way you do now.\n\n\n<a href='https://www.codereviewhub.com/meteor/guide/pull/226?mark_as_completed=1'><img src='http://www.codereviewhub.com/site/github-mark-as-completed.png' height=26></a>&nbsp;<a href='https://www.codereviewhub.com/meteor/guide/pull/226?approve=1'><img src='http://www.codereviewhub.com/site/github-approve.png' height=26></a>&nbsp;<a href='https://github.com/meteor/guide/pull/226'><img src='http://www.codereviewhub.com/site/github-refresh.png' height=26></a>\n<a href='#crh-end'></a>"},"comment":{"url":"https://api.github.com/repos/meteor/guide/issues/comments/190482405","html_url":"https://github.com/meteor/guide/pull/226#issuecomment-190482405","issue_url":"https://api.github.com/repos/meteor/guide/issues/226","id":190482405,"user":{"login":"tmeasday","id":132554,"avatar_url":"https://avatars.githubusercontent.com/u/132554?v=3","gravatar_id":"","url":"https://api.github.com/users/tmeasday","html_url":"https://github.com/tmeasday","followers_url":"https://api.github.com/users/tmeasday/followers","following_url":"https://api.github.com/users/tmeasday/following{/other_user}","gists_url":"https://api.github.com/users/tmeasday/gists{/gist_id}","starred_url":"https://api.github.com/users/tmeasday/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmeasday/subscriptions","organizations_url":"https://api.github.com/users/tmeasday/orgs","repos_url":"https://api.github.com/users/tmeasday/repos","events_url":"https://api.github.com/users/tmeasday/events{/privacy}","received_events_url":"https://api.github.com/users/tmeasday/received_events","type":"User","site_admin":false},"created_at":"2016-03-01T01:29:38Z","updated_at":"2016-03-01T01:29:38Z","body":"Symlinks are problematic on windows, it's true.\r\n\r\nThe approach we've used is git submodules within the app for shared code. We are yet to do it in any significant way for modules so far, but I don't see any reason why it wouldn't work."}}