Skip to content

Conversation

@divisio74
Copy link
Contributor

@divisio74 divisio74 commented Sep 18, 2025

Pierre115's Home Page

This PR adds a simple Home page.
Please feel free to give me feedback !

Preview

image

Copy link
Contributor

@leohhhn leohhhn left a comment

Choose a reason for hiding this comment

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

Cool. I left some more comments - please see them below.

@leohhhn leohhhn changed the title [ADD] : Pierre115's home - For testing pr on the repo feat: add Pierre115's home Sep 22, 2025
@leohhhn
Copy link
Contributor

leohhhn commented Sep 22, 2025

Also renamed the PR ^

@divisio74
Copy link
Contributor Author

Thank you for your feedback. The code has been corrected and updated !

Copy link
Contributor

@leohhhn leohhhn left a comment

Choose a reason for hiding this comment

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

A bit more work needed; check out my comments below.

When you work on things that the reviewers requested, you should make a commit for each fix, and then post the commit link in the related conversation. this makes it a lot easier for the reviewers. Check out some of the PRs on the monorepo to see examples

@leohhhn
Copy link
Contributor

leohhhn commented Oct 2, 2025

Can you please check the CI, and also reply to comments?

@leohhhn
Copy link
Contributor

leohhhn commented Oct 3, 2025

Ci still failing!

@leohhhn leohhhn merged commit 0c23b1c into gnoverse:main Oct 21, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants