Skip to content

Assignment 1#1

Open
BGoulem wants to merge 8 commits intomainfrom
assignment-1
Open

Assignment 1#1
BGoulem wants to merge 8 commits intomainfrom
assignment-1

Conversation

@BGoulem
Copy link
Copy Markdown
Owner

@BGoulem BGoulem commented Apr 1, 2026

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

This assignment was creating tables from SQL from multiple data tables, reflecting on ethics of data repositories, and creating a logical data model.

What did you learn from the changes you have made?

I continue to be very frustrated by the git pull process. I spend a disproportionate amount of time uploading the git repo to git hub. It has been very challenging.

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

Again this week the approach was very guided by the instructions given and the hints.

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

I had a lot of challenges with git commit. I spent a lot of time reviewing the course material from weeks 1 and 2, and on the internet looking for support for strange git issues.

How were these changes tested?

The sql changes were tested in DB Browser for SQLite. I tested changes using a trial and error approach, many times when something wasn't working, there was a misspelt title or missing letters.

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

Checklist

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

Copy link
Copy Markdown

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

Very well done! 30/30

@mrpotatocode
Copy link
Copy Markdown

@BGoulem, a very thoughtful section 4

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