Skip to content

Update README.md #561

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update README.md #561

wants to merge 1 commit into from

Conversation

Suhaib3100
Copy link

Portfolio Author

Portfolio Link?

GitHub Link:

Tech Stack

Copy link

vercel bot commented Apr 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
portfolio-ideas ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 10, 2025 2:16pm

Copy link

Hello @Suhaib3100, 🔥 thanks for raising a pull request in this project. Now, sit back and drink some coffee while we review this.

@Suhaib3100
Copy link
Author

Portfolio Author: Suhaib SZ
Portfolio Link: ceo.protool.co.in
GitHub Link:Mentioned
Tech Stack: Nextjs

Copy link
Collaborator

@israelmitolu israelmitolu left a comment

Choose a reason for hiding this comment

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

Hi @Suhaib3100
Thanks for adding your portfolio. However, the screenshot is quite large (1.2MB). We have many images in the repo, so we try to make them as small as possible. Ideally less than 150kB.

Kindly compress it using squoosh, and then reupload.
Also, kindly move your portfolio to the bottom of the list

Thanks for your understanding :)

@IgorKowalczyk
Copy link

Hi, unfortunately you are not the original author of this portfolio

The original repository: https://github.com/IgorKowalczyk/igorkowalczyk.dev
Website: https://igorkowalczyk.dev

My portfolio is open source so that people do interesting things with it and not steal it, I would ask you to at least add credits to who the original author is. Stealing someone else's design and signing it as your own is not right.

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