New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add document on backend code organization #6853
Add document on backend code organization #6853
Conversation
Nice work @sheridancbio! This is really useful. One minor thing is the flow of how one would get here from the README.md . There is a link to the documentation already but maybe it's good to add a line that the architecture and folder structure are described there with links to the relevant sections? |
docs/Code-Organization.md
Outdated
which define the code components and modules which make up the backend. The pom.xml | ||
files also list dependencies on external open source libraries, and dependencies between cBioPortal Maven modules. | ||
They also give instructions for how to build and package code components into a | ||
deployable format. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this - very useful for people not familiar with maven projects
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, looks great and a long needed addition to the docs. I think we need to include/link some of this information into the README.md file of the repos. We can work on that in a separate PR.
9147a44
to
2203efe
Compare
- general page describing existing maven modules - figure showing modules and dependencies - add links in README.md
2203efe
to
9c98a0d
Compare
deleting request for review from @averyniceday in order to merge now |
Describe changes proposed in this pull request:
general page describing existing maven modules
figure showing modules and dependencies
The commit log is comprehensible. It follows 7 rules of great commit messages. For most PRs a single commit should suffice, in some cases multiple topical commits can be useful. During review it is ok to see tiny commits (e.g. Fix reviewer comments), but right before the code gets merged to master or rc branch, any such commits should be squashed since they are useless to the other developers. Definitely avoid merge commits, use rebase instead.
Any screenshots or GIFs?
If this is a new visual feature please add a before/after screenshot or gif
here with e.g. GifGrabber.