-
Notifications
You must be signed in to change notification settings - Fork 0
Fixing minor issues and add some missing repos #28
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -56,7 +56,7 @@ echo ":" | |||||
| # Check if the SRC_DIR directory does not exist | ||||||
| if [ ! -d "$SRC_DIR" ] | ||||||
| then | ||||||
| echo ": ERROR: Directory $SRC_DIR DOES NOT exists." | ||||||
| echo ": ERROR: Directory $SRC_DIR DOES NOT exists. Clone the repo first!" | ||||||
| exit | ||||||
| fi | ||||||
|
|
||||||
|
|
@@ -101,7 +101,7 @@ if [ $KIND_OF = "Maven" ]; then | |||||
|
|
||||||
| echo ": DASH Analysis > " $ANALYSIS_RESULT_FILE | ||||||
| echo ": DASH Analysis > " $ANALYSIS_RESULT_FILE >> $LOG_FILE | ||||||
| java -jar $DASH_LIB $GREP_RESULT_FILE -summary $ANALYSIS_RESULT_FILE >> $LOG_FILE | ||||||
| java -jar $DASH_LIB $GREP_RESULT_FILE -summary $ANALYSIS_RESULT_FILE -batch 500 >> $LOG_FILE | ||||||
| cp $SRC_DIR/pom.xml $DEST_DIR | ||||||
|
|
||||||
| elif [ $KIND_OF = "Nodes" ]; then | ||||||
|
|
@@ -112,8 +112,9 @@ elif [ $KIND_OF = "Nodes" ]; then | |||||
|
|
||||||
| echo ": DASH Analysis > " $ANALYSIS_RESULT_FILE | ||||||
| echo ": DASH Analysis > " $ANALYSIS_RESULT_FILE >> $LOG_FILE | ||||||
| java -jar $DASH_LIB $JSON_FILE -summary $ANALYSIS_RESULT_FILE >> $LOG_FILE | ||||||
| cp $SRC_DIR/package-lock.json $DEST_DIR | ||||||
| java -jar $DASH_LIB $JSON_FILE -summary $ANALYSIS_RESULT_FILE -batch 500 >> $LOG_FILE | ||||||
| cp $SRC_DIR/package.json $DEST_DIR | ||||||
| cp $SRC_DIR/yarn.lock $DEST_DIR | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
I think this should be |
||||||
|
|
||||||
| elif [ $KIND_OF = "Yarn" ]; then | ||||||
| # Analyse Yarn project | ||||||
|
|
@@ -125,6 +126,7 @@ elif [ $KIND_OF = "Yarn" ]; then | |||||
| echo ": DASH Analysis > " $ANALYSIS_RESULT_FILE >> $LOG_FILE | ||||||
| # Reduce the size of the batch (from 1000 to 500) to avoid a time out with ClearlyDefined | ||||||
| java -jar $DASH_LIB $YARN_FILE -summary $ANALYSIS_RESULT_FILE -batch 500 >> $LOG_FILE | ||||||
| cp $SRC_DIR/package.json $DEST_DIR | ||||||
| cp $SRC_DIR/yarn.lock $DEST_DIR | ||||||
|
|
||||||
| else | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| #!/bin/bash | ||
| ./parseAll.sh "/Users/philippe/git" "/Users/philippe/temp/smartclide/analysis" | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe put I would suggest not to include them as source files - at least not as they are now - because they contain file paths that are specific for your machine. |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| #!/bin/bash | ||
| SRC_REPO="/Users/philippe/git" | ||
| DST_REPO="/Users/philippe/temp/smartclide/analysis" | ||
| ./dashAnalysis.sh "Yarn" "$SRC_REPO/smartclide-td-reusability-theia" "$DST_REPO/smartclide-td-reusability-theia" |
Uh oh!
There was an error while loading. Please reload this page.