You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Sounds good. I just looked at your PR and while the directory structure is interesting I think I would prefer to keep the structure flat, like I have in previous years. I would prefer just having a single child level directory under the repository root named for your session and then putting the content there. For reference take a look at the example folder "0 - Sample" or last years repo: https://github.com/TechConf/CodeMash2014 .
I have found this works well not just for content but also for adding in an entire repository as a subrepository as some people have also done in the past.
If you could refactor your PR to meet the above guidance that would be great or let me know and I can do it. Thanks!
Fixed but not sure I see how it is easier to have subprojects.
Let me know if there is anything else I will try to add even more content
in the run up.
On Dec 31, 2014 8:30 AM, "Joe Kuemerle" notifications@github.com wrote:
Sounds good. I just looked at your PR and while the directory structure is
interesting I think I would prefer to keep the structure flat, like I have
in previous years. I would prefer just having a single child level
directory under the repository root named for your session and then putting
the content there. For reference take a look at the example folder "0 -
Sample" or last years repo: https://github.com/TechConf/CodeMash2014 .
I have found this works well not just for content but also for adding in
an entire repository as a subrepository as some people have also done in
the past.
If you could refactor your PR to meet the above guidance that would be
great or let me know and I can do it. Thanks!
Clearly this isn't going to work with 1 level I tried a certain structure in my commit but we should probably discuss it.
The text was updated successfully, but these errors were encountered: