rename docs-team folder to learning-team about core-notes HOT 7 CLOSED

I agree, many of these notes have been cross linked (in various comments, blog posts, etc). I think it would be totally great to start using the correct terminology going forward, and we could even copy the contents into the correct place, but we shouldn't remove it from docs-team.

I think my concrete suggestion would be:

• Add a top level learning-team folder
• Start adding new content to learning-team
• Add a docs-team/README.md explaining that the folder is legacy and new items are added in learning-team.
• Optionally fail CI if a new file is added to docs-team (helps us to not forget 😛)
• Optionally copy all existing docs-team content into the new folder

What say y'all?

from core-notes.

The context here is that the folder was named before the learning team became the learning team 😉 I have no objection in concept (it would be more consistent) but we’d also lose history in the process of the move. @locks can you explain your reaction here?

from core-notes.

@acorncom what would you mean by "los[ing] history" in this case? if we do it as a normal git move then the history will be maintained from a file perspective.

If we want to represent the fact that the docs team "evolved" into the learning team we should probably only move files after the date it was made official.

I just think it's odd that I'm adding something to the docs-team folder. I am not a member of the docs team, I'm a member of the learning team 😂 and it matters what you name things 👍

from core-notes.

My objection is a recurring one, I am not keen on breaking URLs for what I perceive to be little gain in this instance.

from core-notes.

Also, FWIW, learning-team was coined back in May of 2016, so at least some of the content here predates the newer term.

https://emberjs.com/blog/2016/05/19/introducing-subteams.html

We should decide if the content in the learning-team folder is just content after that date, or if it was "retroactive".

from core-notes.

Made this PR, please leave feedback, can make changes to get this closed! #273

from core-notes.

Closed via #273

from core-notes.