deploy-mdbook icon indicating copy to clipboard operation
deploy-mdbook copied to clipboard

Syntax error: word unexpected (expecting ")")

Open bananaturtlesandwich opened this issue 2 years ago • 11 comments

job ended with failure but there's not enough information for me to know what's wrong https://github.com/bananaturtlesandwich/UE4-Modding-Guide/actions/runs/5989677648/job/16246137933

bananaturtlesandwich avatar Aug 27 '23 07:08 bananaturtlesandwich

Thank you for your issue! However that seems to be an issue with mdbook not deploy-mdbook, we just install the latest mdbook, if you run your project locally with the latest version you should get the same error.

XAMPPRocky avatar Aug 27 '23 07:08 XAMPPRocky

seems to be fine for me and I'm on latest version :/

bananaturtlesandwich avatar Aug 27 '23 10:08 bananaturtlesandwich

works from clean too

bananaturtlesandwich avatar Aug 27 '23 10:08 bananaturtlesandwich

seems to be a linux issue but I remember the workflow doesn't work on any other platform so can't switch https://stackoverflow.com/questions/36725864/syntax-error-word-unexpected-expecting

bananaturtlesandwich avatar Aug 27 '23 10:08 bananaturtlesandwich

Hmm but there's no usage of ( in the code? Could you try an older Ubuntu version and see if that works?

XAMPPRocky avatar Aug 27 '23 16:08 XAMPPRocky

there's only one other one - 20.04 and that also fails https://github.com/bananaturtlesandwich/UE4-Modding-Guide/actions/runs/5989669153/job/16246099138

bananaturtlesandwich avatar Aug 27 '23 17:08 bananaturtlesandwich

We're having the same issue here. Replacing the action with a manual cargo install mdbook --locked; mdbook build succeeds.

celinval avatar Oct 06 '23 21:10 celinval

I'm away this week but I think the solution is probably to just update the action's dependencies and then publish a new bundled version, because this doesn't seem like an issue in the action itself.

XAMPPRocky avatar Oct 07 '23 08:10 XAMPPRocky

bump

GeckoEidechse avatar Feb 03 '24 20:02 GeckoEidechse

I'm away this week but I think the solution is probably to just update the action's dependencies and then publish a new bundled version, because this doesn't seem like an issue in the action itself.

do you need someone to put up a PR ?

rbtcollins avatar Jul 19 '24 21:07 rbtcollins

@rbtcollins Yes that would be greatly appreciated, as it's a lot easier to fit code reviews into my schedule than coding time.

XAMPPRocky avatar Jul 19 '24 23:07 XAMPPRocky