Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

passed the tests #5272

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

passed the tests #5272

wants to merge 9 commits into from

Conversation

rokster112
Copy link

@rokster112 rokster112 commented Feb 3, 2025

❗️ Copy this Checklist to the Pull Request description after links, and put - [x] before each point after you checked it.

  • there 2 BEM blocks card and stars each in their own file
  • SCSS Nesting is used for elements, modifiers and pseudo-classes
  • SCSS Variables are used for main values and placed in a separate file
  • all stars--N modifiers work as expected (Highlight first N stars)
  • Code follows all the Code Style Rules ❗️

@rokster112
Copy link
Author

This is 2nd project where after doing pull requests it breaks my project. I cannot run the server anymore with npm start. I get in the browser 404 Page Not Found. So, again, I cannot complete the task.

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