Skip to content

Copy constructors and partial constructor deprecation

Luc Everse requested to merge copy-ctors into development

Description

Introduces copy constructors to Assignment, Submission, and any dependencies of these classes. To reduce the number of active constructors, partial constructors are deprecated from now on.

Changes

  • Deprecated Assignment(String), Assignment(String, int, int), and Assignment(int, int)
  • Deprecated Submission(String, String, String)

Additions

  • Added copy ctor for Assignment
  • Added copy ctor for Submission
  • Added copy ctor for MetricSetttings
  • Added copy ctor for SubmissionPipelineLog

Test and Review

To be filled in by the reviewers

  • All of the methods are commented to expectation

  • The methods are tested to satisfaction

  • There are no unnecessary files present in the MR

  • The continuous integration has no problems with the MR

  • The MR is filled in as requested (including labels, milestones, and reviewers)

  • The documentation is up-to-date

  • All nullable parameters are marked as such

Edited by Erik Oudsen

Merge request reports