Merge pull request #57 from ChicoState/devops

Updated devops with instructions for volume bind mount and default repo copied into container's working directory

Dockerfile

@@ -21,6 +21,9 @@ RUN apt-get update && \
 COPY . ./
 RUN dos2unix *
 
+# Load repo into image
+COPY . ${WORKDIR}
+
 # Run static analysis
 CMD pmccabe *.cpp -v && \
     cccc --lang=c++ --outdir=report *.cpp

README.md

@@ -33,4 +33,8 @@ To run static analysis in the dev container:
 docker run -it cpp-analysis
 ```
 
-Or to run it with binding to changes to source code (but not the container configuration) use the above command with the added options `--mount type=bind,source="$(pwd)",target=/usr/src` *before* the container name.
+To bind file changes between the host and container:
+
+```
+docker run -v "$(pwd)":/usr/src -it cpp-analysis
+```

dictionary.cpp

@@ -17,7 +17,6 @@ void dictionary::load_dictionary(std::string file){
     std::vector <std::string> temp;
 
     if( read.is_open() ){
-
         std::string word;
         while( getline(read,word) ){
             //convert every word to UPPER CASE
@@ -26,8 +25,6 @@ void dictionary::load_dictionary(std::string file){
             }
            temp.push_back(word);
         }
-
-
         if( temp.size() > 0 ){
             words = temp;
             dictionary_file = file;