Skip to content

Data Ingest Script Updated for Part 1 of assignment#1

Open
obedmortey wants to merge 4 commits intomainfrom
assignment
Open

Data Ingest Script Updated for Part 1 of assignment#1
obedmortey wants to merge 4 commits intomainfrom
assignment

Conversation

@obedmortey
Copy link
Owner

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

I am completing the script by adding a few codes based on the comments in the assignment.sh file.

What did you learn from the changes you have made?

I learned how to create and remove files and folders, copy and change directories.

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

Regarding comment (8), I was thinking of creating a text file then listing the files in the subfolders in the created file:
touch data/inventory.txt
ls data/processed/* > inventory.txt

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

I was finding it difficult in submitting the assignment and also making my script run in any directory.

How were these changes tested?

I tested my script in multiple folders and paths and it worked well.

Checklist

  • [ x ] I can confirm that my changes are working as intended
    • [ x ] I can confirm that I tested my script in multiple directories.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Autograder results

question status comment
Part 1 - Q1
Part 1 - Q2
Part 1 - Q3
Part 1 - Q4
Part 1 - Q5
Part 1 - Q6
Part 1 - Q7
Part 1 - Q8
Part 2 coworker-changes branch not found in commit history

Please address the issues listed above.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Autograder results

question status
Part 1 - Q1
Part 1 - Q2
Part 1 - Q3
Part 1 - Q4
Part 1 - Q5
Part 1 - Q6
Part 1 - Q7
Part 1 - Q8
Part 2

Copy link

@eric-j-sanders eric-j-sanders left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The assignment is 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