Skip to content

Conversation

@Jushah5
Copy link
Owner

@Jushah5 Jushah5 commented Nov 25, 2024

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

Added commands, made directories, hanged the script, moved content from directories, saved the work

What did you learn from the changes you have made?

learnt a few steps on how commands in work in VSC

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

The instruction documents that had the screenshots were really helpful to understand in what order to steps and how should the end result look like.

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

The softwares are a bit more complicated than I expected, took me a very long time to undetstand how they interact with each other. A lot of errors were mainly solved by a friend who is also taking the course and Chat GPT.

How were these changes tested?

As recommended I tried a few changed in the terminal

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

Checklist

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

Copy link

@anjali-deshpande-hub anjali-deshpande-hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please Enable 'Actions' in workflows and see how the autograder responds to your answers?
Screenshot 2024-11-25 150428
Please see the above steps in instructions

@anjali-deshpande-hub
Copy link

  1. rename the branch as 'assignment'. Check if autograder runs on it.
  2. The 'cd ./data/processed' in Question 4 will fail because there is no 'processed' dir. Clue: Use a mkdir option to create the directories recursively
  3. The 'cd ./data/processed/server_logs' in Question 5 is incorrect because the commands following that command will fail since you are in incorrect dir.
    Usage of 'cd' command in shell scripts is not advisable, because you generally don't want to change into another directory after running the shell script.
  4. Complete the Part 2 of the assignment. See instructions

Copy link

@anjali-deshpande-hub anjali-deshpande-hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make review changes suggested above and re-submit. No need to make new PR. The same PR will show the updates made to the 'assignment' branch

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.

2 participants