Skip to content

Conversation

@PSulf
Copy link
Owner

@PSulf PSulf commented Apr 18, 2025

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

Finalized assignment.sh by adding Bash commands to organize logs into folders, copy necessary files, remove IP files, and create an inventory.

What did you learn from the changes you have made?

Learned how to use Bash commands to automate workflows, including wildcards, file operations, conditional checks, and proper Git/GitHub workflow (branching, committing, pushing, and pull requests).

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

Initially thought about testing in the main directory but later used a clean test setup (assignment_test_clean) to properly validate everything.

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

Faced Git config errors, merge conflicts, and a missing PR to the correct fork branch. All fixed by setting username/email, manually resolving conflicts, and ensuring the PR was made from assignment to PSulf/main.

How were these changes tested?

Tested using bash assignment.sh in a clean directory. Verified that all folders were created, log files copied or removed as expected, and the final inventory was accurate.

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

Not applicable.

Checklist

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

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

@dtxe
Copy link

dtxe commented Apr 19, 2025

Well done! 🎉

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