Folder structure for packages and Java about startup-os HOT 6 OPEN

My personal opinion:

It is not confusing to put separate language files in separate roots. In fact, it's the opposite. It helps keep things organized and makes it easier to find the code. google3 and most multi-language github projects I've seen do this.

That said, I don't know enough about bazel to have a strong, informed opinion here. But I do think most Java developers will find the existing layout confusing.

from startup-os.

Do you mean that,
com.google.startupos.tools.reviewer.aa.AaTool
is in folder tools/reviewer/aa instead of java/com/google/startupos/tools/reviewer/aa and tests in javatests/com/google/startupos/tools/reviewer/aa?

from startup-os.

Yes, that's what I meant.

from startup-os.

Right, so we've actually had some discussions on that, here:
#30
The idea is that Bazel is a multi-language built tool, so if a repo has multiple projects, and say some project has java, scala, c++, python and proto (quite conceivable), it's confusing to have each in a separate root folder, i.e:
java/com/google/startupos/tools/reviewer/aa
javatests/com/google/startupos/tools/reviewer/aa
and then similarly,
<language>/com/google/startupos/tools/reviewer/aa
<language>tests/com/google/startupos/tools/reviewer/aa
for every language.

So the idea is that we organize folders based on projects, and keep project-related files together.

from startup-os.

How do we move forward with this issue?

from startup-os.

I'm not sure what you mean by a root here. google3 certainly does separate java, C++, etc. into separate directory structures and google3/java follows java conventions on package organization. In this case google/startup-os is the equivalent of google3.

from startup-os.