-
Notifications
You must be signed in to change notification settings - Fork 256
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
Need to add Deploy status badge in README.md #766
Comments
I suppose there are some inconsistency with the deployment command. Because the npm i command throws error. |
@Pradumn-18 could you share a screenshot. |
Yeah sure, I am attaching the log file of Netlify deployment build for better insights. The package installation is fine now. I figured that the package installation works fine with lower NodeJs version like 14.x, 12.x but gives error on 16.x. Now I face error in "npm run develop" command mentioned in README.md of the repository. Please take a deep look into it. |
Tried on Travis-CI This is the failing one: |
@raghunath-r-a @abhir9 you can assign me this issue. Just specify the final testing platform. I prefer going with Travis or as @Pradumn-18 said Netflify. |
@AmreshSinha I guess Travis CI will not be preferable because their you would not be able to deploy the site. It will start executing the build but at last this is basically a site to be hosted, which will keep running in your build if it passes.I myself tried same on Github Action which is equivalent to Travis CI. Moreover could you please share the logs of build that you claim is passing. |
@Pradumn-18 and the fail error log: Ig the thing is with node version. It's working fine in node v14.x or node v12.x (as in Travis-CI) |
@AmreshSinha Yes I agree to that, it fails with node version 16.x as I have also pointed out same in my prior comment. |
@AmreshSinha @Pradumn-18 Thanks for highlighting this issue. This issue is due to an older version of Gatsby being used. We've created a new issue for the same: #789 Meanwhile, adding a status badge in README.md has been addressed by another pull request. (For your reference: #780) |
@Pradumn-18 Seems like gatsby version used in this portal is 2.x.x, which is not supported in node v16.x, kindly raise the PR in support to node version either For more details checkout this issue #789 |
Though the PR seems to have merged but unfortunately I am still not able to the badge for Netlify deployment. Could you please look into that. |
@mohammed786 Sure would do that. |
Wish to work on that |
@dontech09 Thanks for your interest in fixing this. But this has been already addressed by this pull request: #792 |
Please add the Deploy status badge in README.md
Status badges are visual representations of a site's status, served as image files that can be added to the site repository's READMEs, documentation, or any other web page. Please follow this URL
The text was updated successfully, but these errors were encountered: