-
Notifications
You must be signed in to change notification settings - Fork 379
gh-pages deployment action failing #30
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
Comments
The error says
The root of the problem is here
This is not a valid GitHub ID. Seems like we also have to check githubId field before merging a pr.
Yarn executes tasks in parallel and much faster than npm especially while installing. |
@avats-dev Add what is a correct GitHub ID in the guide. Additionally, I found many are not changing |
@Janglee123 yeah, I'll add a reference as to what a GithubId is in the guide. Also thanks for the tip on yarn. Apart from this, in #32 , first his checks were failing due to wrong bio. Then he edited someone else's bio, and then checks passed, how? Is it like if any of the previous PRs are invalid, next ones will get stuck too? Also, @garg3133 :
What's wrong in his bio? I'm not getting this. |
Yes, GitHub actions is checking all the existing json files also in each PR.
Actually, the checks in #31 had already failed due to this file before its bio was fixed in #32. So, it had already been fixed when I edited it and there was nothing wrong in this file. But as I didn't read the changes in GitHub actions, I thought the bio needs to be completely changed. So, I changed it. |
Its shouldn't be. I have to check this. Okay, there are many problems with Merge Bot now! Have to fix them. |
Since some time (to be exact, since last two merges), automatic deployment of gh-pages is failing.
The error from logs look like this:
Possible reasons:
Possible Resolution
I don't know much but what I found is that following methods might solve this (not sure):
Reference : vuejs/vue-cli#439
or
These might help resolve package issues if there would be any.
Also @Janglee123 why are we not using npm instead of yarn?
The text was updated successfully, but these errors were encountered: