Skip to content

Assignment shell - #1

Merged
bnguven merged 5 commits into
mainfrom
assignment_shell
Oct 3, 2026
Merged

bnguven merged 5 commits into
mainfrom
assignment_shell

Conversation

@bnguven

@bnguven bnguven commented Sep 30, 2026

Copy link
Copy Markdown
Owner

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

Made a new directory, organized raw data into folders, removed files with identifying IP information, finally created an inventory to keep all the processed data files.

What did you learn from the changes you have made?

I learned shell commands like mkdir, mv, ls, cp, rm, touch, find. I learned how to create a branch, add, commit, and push to git. I also learned about merging and resolving conflicts.

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

Not quite.

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

I had forgotten that I can use command options with dash (-). I googled and used a bit of AI to figure it out.

How were these changes tested?

I used git status and git log --oneline and also looked at what the script produced locally on my device.

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

Checklist

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

@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.

@Dmytro-Bonislavskyi Dmytro-Bonislavskyi 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.

Well done!

@bnguven
bnguven merged commit 7e7392a into main Oct 3, 2026
3 checks passed
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.

4 participants