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

replace img with nextjs Image component #20

Merged
merged 3 commits into from
Aug 21, 2024

Conversation

RafaelCaso
Copy link
Contributor

Description

replace img tag with NextJs Image to avoid CI pipeline issues

Copy link

vercel bot commented Aug 16, 2024

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

Name Status Preview Comments Updated (UTC)
batch8-buidlguidl-com-nextjs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 17, 2024 0:50am

@derrekcoleman
Copy link
Collaborator

Thank you, @RafaelCaso.

The build is failing for a new reason:

./app/builders/0xfb8b972b0D1cF9BB242A40F7AEA8E9a1bF89E0E3/page.tsx
3:33 Warning: Delete · prettier/prettier
5:11 Warning: Delete · prettier/prettier
55:20 Warning: Insert ; prettier/prettier

The numbers at the beginning of each statement refer to the line and character location within the file. Could you do another commit to fix those?

Do you have the Prettier plugin installed in your IDE? It should catch these things for you.

@RafaelCaso
Copy link
Contributor Author

I do (have prettier) but it looks like that wallet address is someone else's (looks like 0xzeze's).

Let me know if there's still something I can do to fix this.

@derrekcoleman derrekcoleman merged commit d907403 into BuidlGuidl:main Aug 21, 2024
2 checks passed
ruhneb2004 pushed a commit to ruhneb2004/batch8.buidlguidl.com that referenced this pull request Aug 22, 2024
* replace img with nextjs Image component

* Hotfix: fix linting issue
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.

2 participants