apluslms / radar

Web service for analyzing program code similarity
4 stars 10 forks source link

Development container environment #20

Closed Gehock closed 3 years ago

Gehock commented 3 years ago

Description

This pull request adds a dev/test environment similar to run-aplus-front+aplus-manual using docker and docker-compose. The PR is marked as a draft because the feature is still very much work-in-progress and requires some clean-up before merging, but feedback and testing are already welcome at this stage. The first commit ("Add initial files from run-aplus-front") contains a squashed history of run-aplus-front, on top of which this feature has been built.

Have you updated the README or other relevant documentation?

There is a separate documentation file at docker/README.md.

Is it Done?

Gehock commented 3 years ago

Documentation still requires some small changes but otherwise this is more or less ready

Gehock commented 3 years ago

@Mankro This PR is more or less ready, I could use some small feedback on whether the documentation is understandable enough, but otherwise this is ready to merge if it looks good to you

Gehock commented 3 years ago

Shall we squash the git commits before merging?

I would prefer creating a merge commit instead. I tried to reorganise the commit history into a set of logical changes, so the commit history itself now acts as some documentation of the development process. If nothing else, I think it would be important to make a distinction between the first commit, which just contains a squashed version of run-aplus-front, and the rest of the commits that actually introduce the radar-specific functionality.