Skip to content

Conversation

@AshiqNoor-S
Copy link
Contributor

@rohin079 I have fixed the issues related to build

I have run the build command locally and it is successful, attached below is a screenshot for your reference

ss

@netlify
Copy link

netlify bot commented Oct 6, 2023

Deploy Preview for upes-gdsc ready!

Name Link
🔨 Latest commit 1bd6707
🔍 Latest deploy log https://app.netlify.com/sites/upes-gdsc/deploys/652f95279d027300085f0330
😎 Deploy Preview https://deploy-preview-33--upes-gdsc.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100
Accessibility: 84
Best Practices: 100
SEO: 92
PWA: 70
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@rohin079
Copy link
Member

rohin079 commented Oct 7, 2023

@AshiqNoor-S Kindly look at the build preview logs to find out why the build is failing, correct it and then try to raise another commit until all checks are passed

@rohin079
Copy link
Member

rohin079 commented Oct 7, 2023

@AshiqNoor-S Kindly delete these lines of code since they aren't used and commit again:
image

@AshiqNoor-S
Copy link
Contributor Author

@AshiqNoor-S Kindly delete these lines of code since they aren't used and commit again: image

Done

@rohin079
Copy link
Member

@AshiqNoor-S Hey, the build deploy is still failing, please view the build logs and view why the PR is failing and commit again. Thanks

@AshiqNoor-S
Copy link
Contributor Author

@rohin079 okay

@AshiqNoor-S
Copy link
Contributor Author

@rohin079 I actually went through the code, I cant find anything wrong
image
I also tried building it locally and I am not getting any error

@rohin079
Copy link
Member

@AshiqNoor-S As i can see in your shared screenshot, you have warnings by eslint while trying to build the code, the build checks may also fail due to these warning, Try fixing them such as removing the completed and setcompleted identifier. Also the build log says you have conflicting files like app.jsx and Event.jsx, try to resolve the conflict and commit again.

@AshiqNoor-S
Copy link
Contributor Author

@rohin079 I have removed all the used states and variables
image
The build now shows no warnings

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