eloots / cmt-2

Course Management Tools 2.0

Course Management Tools - 2.0.0

POC for a new Course Management Tools implementation approach

This repo has been archived and the code has been merged into the course-management-tools repository

cmt-2's Issues

printError => printErrorAndExit usage

printError is in one file, printErrorAndExit is in another - merge?

Add scalafmtCheckAll to the CI process

I just ran scalafmtAll before committing a PR and there are a ton of formatting changes clouding the changed files... if we automatically fail a build on incorrect formatting that'll help force us to be more disciplined and keep these PRs focussed.

BDD functional test specifications

• Create a new test SBT module
• Add initial BDD scenarios (not yet implemented)

CMTC windows compatibility check

The cmdc and cmta applications utilise sbt.io for more file system related operations. Some exceptions exist though, for example in the implementation of the "don't touch files" functionality.

We need to check this on a Windows system and potentially add checks in CI that systematically check this

Validation of cli input types

Change printError to printErrorAndExit

Method printError not only prints a given error message to stdout (in RED), but it also does aa System.exit().

It should be renamed to printErrorAndExit.
A new method printError should be added that prints a given message to stderr without exiting the program.

All admin commands to return Either[String, Unit]