Skip to content

Assignment 1#1

Closed
ericamika wants to merge 8 commits intomainfrom
assignment-1
Closed

Assignment 1#1
ericamika wants to merge 8 commits intomainfrom
assignment-1

Conversation

@ericamika
Copy link
Owner

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

updating, merging and testing script

What did you learn from the changes you have made?

I learned about updating data through bash

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

N/A

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

I faced several challenges. Difficulties with successful commits - overcame by resolving a divergent branch and fixing the way I had saved the file. I had difficulties executing the script to provide the correct echo command at the end, but was advised that as long as other steps had been followed correctly and I could carry out my pull request, it was probably fine.

How were these changes tested?

I tested each separate command as I went to make sure they were working. I double checked the organization of data and the removal of particular data.

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

Checklist

  • [YES] I can confirm that my changes are working as intended

@github-actions
Copy link

Autograder results

question status comment
1 data directory does not exist
2 rawdata not moved to data/raw
3 ls command not run
4 Missing directories: data/processed/server_logs, data/processed/user_logs, data/processed/event_logs
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 data/inventory.txt does not exist
9

@ericamika ericamika closed this Jan 11, 2025
@ericamika ericamika deleted the assignment-1 branch January 11, 2025 22:50
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.

2 participants