Skip to content

TITLE: UofT-DSI | Shell - Assignment 2 - #1

Open
Biancalli wants to merge 5 commits into
mainfrom
assignment_shell
Open

Biancalli wants to merge 5 commits into
mainfrom
assignment_shell

Conversation

@Biancalli

Copy link
Copy Markdown
Owner

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

I have added shell scripts to automate file processing and created a script to move files.

What did you learn from the changes you have made?

I learned how to use loops, conditionals, and arguments in shell scripts to automate tasks and manage files.

Was there another approach you were thinking about making?

I considered using Python for file processing but stuck with shell scripting as per the assignment requirements.

Were there any challenges?

I faced challenges when handling edge cases like files with no extensions or hidden files. I resolved these issues by adding checks within the script.

How were these changes tested?

I tested the scripts on a sample directory with various file types and used different test cases to ensure the scripts handled all edge cases properly.

Checklist

  • I can confirm that my changes are working as intended
  • I have tested the script with different file types, and it works as expected

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

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

Approved

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