Skip to content

mustafa - #15

Open
homsi6313-art wants to merge 1 commit into
HackYourAssignment:mainfrom
homsi6313-art:main
Open

homsi6313-art wants to merge 1 commit into
HackYourAssignment:mainfrom
homsi6313-art:main

Conversation

@homsi6313-art

Copy link
Copy Markdown

No description provided.

@github-actions

github-actions Bot commented Apr 1, 2026

Copy link
Copy Markdown

📝 HackYourFuture auto grade

Assignment Score: 0 / 100 ✅

Status: ✅ Passed
Minimum score to pass: 0
🧪 The auto grade is experimental and still being improved

Test Details

@rafaelhdr rafaelhdr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The task 1 is good. Just requiring small adjustments. But you forgot the task 2.

Comment thread task-1/queries.sql

-- Question 6
INSERT INTO books (title, published_year, genre, author_id)
VALUES ('My First Book', 2024, 'Adventure', 6);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Are you sure the id is 6? Because there are so many authors in the original...

Comment thread task-1/queries.sql

-- Question 8
DELETE FROM books
WHERE title = 'My First Book';

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You could be more strict, because it was asked "Make sure your query targets only that specific row."

If another author also have the same book title, it would be deleted.

You could try to use the author id, or the book id to be sure. The book id would be better.

@rafaelhdr rafaelhdr added Reviewed This assignment has been reivewed by a mentor and a feedback has been provided and removed Review in progress labels Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Reviewed This assignment has been reivewed by a mentor and a feedback has been provided

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants