Skip to content

Completed Assignment - Kat Nov 23 - 4PM#1

Merged
Malekikh merged 1 commit intomainfrom
assignment
Nov 23, 2024
Merged

Completed Assignment - Kat Nov 23 - 4PM#1
Malekikh merged 1 commit intomainfrom
assignment

Conversation

@Malekikh
Copy link
Owner

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

This is a completed version of my assignment and trying to use the pull request.

What did you learn from the changes you have made?

I learned different commands and how to test them on my script.

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

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

Yes, I was able to resolve the issue after following the assignment instructions. Initially, I was getting an error stating that I had not merged the changes properly (Part 2 of the assignment), even though I had accepted my current changes. To address this, I manually opened the file, as it showed merge conflict markers like <<< HEAD. Specifically, on line 60, there were conflicting changes marked with <<<<<<< HEAD and >>> left. I resolved this by accepting my changes, and the merge issue was fixed.

Additionally, I encountered an issue with rawdata.zip already existing. Initially, I added the unzip rawdata.zip command, as I was unable to see the .log files. However, I later realized that every time I ran my script, the rawdata.zip file was being unzipped again. To fix this, I removed the unzip command, and now the script runs without any issues.

How were these changes tested?

I used bash assignment.sh to check if my script was working.

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

I am still not sure how to use git status log

Checklist

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

@Malekikh Malekikh merged commit 0b03576 into main Nov 23, 2024
Malekikh added a commit that referenced this pull request Nov 29, 2024
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.

1 participant