Skip to content

Coworker changes#3

Open
cdanss wants to merge 4 commits intomainfrom
coworker-changes
Open

Coworker changes#3
cdanss wants to merge 4 commits intomainfrom
coworker-changes

Conversation

@cdanss
Copy link
Copy Markdown
Owner

@cdanss cdanss commented Apr 11, 2026

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

I modified a shell script that creates a folder, moves files/folder and generates a report at the end

What did you learn from the changes you have made?

How to use shhell

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

I used find to create txt file

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

Yes, I was initially using ./ without realizing that the first part of the code was already changing the working directory to newproject

Another challenge is that I forgot to do the part 2 of the assignment. I had to cancel my pull request, finish the task with the merge and send a new pull request

How were these changes tested?

bash assignment.sh I and checked the inventory and the folder content

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

Checklist

  • I can confirm that my changes are working as intended

@vishnouvina
Copy link
Copy Markdown

Please enable github actions on this PR too for 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.

3 participants