Skip to content

Assignment#1

Open
capa-detated wants to merge 8 commits intomainfrom
assignment
Open

Assignment#1
capa-detated wants to merge 8 commits intomainfrom
assignment

Conversation

@capa-detated
Copy link
Owner

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

Adding in new bash commands to complete the tasks listed. Also manually merged with a coworker's version.

What did you learn from the changes you have made?

I learned to incorporate bash commands into a .sh file, basics of using git and github (both via terminal and on website), and basic use of the terminal and Visual Studio Code.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

While I mostly used the github website interface to take care of the git processes, I also tried to perform some of these functions in terminal to make sure I understand how to use both.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

The biggest challenge I faced was accidentally doing all the work on the 'main' branch rather than the 'assignment' branch, so I ended up deleting the entire forked repo from my account and restarting the task from within the appropriate branch. I also struggled with making sure the current working directory was set appropriately before each set of bash commands was executed, which ended up in many 'file/directory does not exist' error messages.

How were these changes tested?

Changes were tested by running the entire script via terminal and checking the output, which in this case meant verifying that all relevant files were created and/or moved into appropriate folders, while other files were deleted. I did this inside the file explorer UI in VScode.

A reference to a related issue in your repository (if applicable)

Checklist

  • I can confirm that my changes are working as intended

@github-actions
Copy link

Autograder results

question status comment
1
2 rawdata not moved to data/raw
3
4
5
6
7
8
9

…hat mv rawdata to data/raw was not performed correctly, so fixed this to move the entire directory rather than only the contents, as it was doing before.
@github-actions
Copy link

Autograder results

question status comment
1
2
3
4
5 No server log files in data/raw
6 No user log files in data/raw; No event log files in data/raw
7
8
9

@github-actions
Copy link

Autograder results

question status
1
2
3
4
5
6
7
8
9

@ananyajha2000
Copy link

Assignment marked complete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants