Skip to content

Assignment 1 due Sept. 30 (extension granted until Oct. 4) - #1

Merged
tselot-t merged 1 commit into
mainfrom
assignment_shell
Oct 13, 2026
Merged

tselot-t merged 1 commit into
mainfrom
assignment_shell

Conversation

@tselot-t

@tselot-t tselot-t commented Oct 4, 2026

Copy link
Copy Markdown
Owner

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

A new zip containing raw data was opened and organized into processed data subfolders. Some files were deleted because they contained sensitive data.

What did you learn from the changes you have made?

I learned how to unzip files and organize them, which includes creating new directories, moving files, and deleting files.

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

I was wondering whether I could make the code more concise, but I'm not sure how.

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

I ran into a few challenges that required some troubleshooting. First, I figured out that I didn't need to run the whole script on VS Code each time to test my code - I could run each line on the terminal and even use the fake data and directories created in class. I learned how to attach multiple wildcard commands to find specific files (e.g. server.log). I learned how to use the recursive function for the ls command.

How were these changes tested?

I tested the changes using the ls command.

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

Checklist

  • I can confirm that my changes are working as intended

@mbonsma

mbonsma commented Oct 9, 2026

Copy link
Copy Markdown

Hi Tselot, can you enable automatic workflows under the "Actions" tab so that the autograder can be used?

@tselot-t tselot-t closed this Oct 9, 2026
@tselot-t tselot-t reopened this Oct 9, 2026
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown

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

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Autograder checks failed. Please address the issues listed in the results comment.

@mbonsma

mbonsma commented Oct 13, 2026

Copy link
Copy Markdown

Hi Tselot, you're just missing Part 2 - after completing that, you'll be done.

@tselot-t
tselot-t merged commit e4dbd35 into main Oct 13, 2026
2 checks passed
@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Autograder checks failed. Please address the issues listed in the results comment.

@github-actions

Copy link
Copy Markdown

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 ✅

All autograder tests passed. This submission is approved by the autograder.

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