search

nvimdev / guard-collection

collection the tools config for guard
MIT License
31 stars 9 forks source link

Some tests work locally but not on CI #29

Closed xiaoshihou514 closed 11 months ago

xiaoshihou514 commented 11 months ago

And there's quite a few. No idea why :sweat_smile:

barrett-ruth commented 11 months ago

Trying to figure this out, but have never used vusted before - how can I run tests locally? Getting this error now.

image
xiaoshihou514 commented 11 months ago

@barrett-ruth check out .github/workflows/test.yml

https://github.com/nvimdev/guard-collection/blob/main/.github/workflows/test.yml#L51-L60

barrett-ruth commented 11 months ago

Sorry if I'm being dumb... but I sat that? How can I run the tests locally, that workflow command seems to yield an error:

image
xiaoshihou514 commented 11 months ago

Ah, you had the exact same problem as I had 😂 https://github.com/notomo/vusted/issues/12

barrett-ruth commented 11 months ago

@xiaoshihou514 after trying and failing to fix this pr for one tool (namely latexindent), I realize this issue would best be debugged using act. For example, on my system, latexindent couldn't simply be run and work immediately - I had to use sudo cpan -i File::HomeDir for it to work. Point being, I think the tests are right, but there's a lot of system-specific issues in the CI that must be addressed.

For example, if you look at the log of my PR, latexindent still doesn't run despite me trying the aforementioned fix. Furthermore, it doesn't have any real error logs in the output, meaning (I think) the command itself is just erroring out and not starting up. This is nearly impossible to debug, but a tool like act would definitely help out. I also fixed up some minor things in the workflow file as well.

I have interviews and exams coming up so I won't have that much time to dedicate to this issue (namely manually debugging and fixing every single individual package). I'll do my best to work on it this week but would also appreciate it if you checked out the pr.

xiaoshihou514 commented 11 months ago

@barrett-ruth Thanks for doing all these :+1: I've also been distracted by school stuff (and scripting in my dots) lately, but I will try to check it out during weekends. act seems like a life saver, previously I've been waiting five minutes to run one debug session. I will try to sort it out in the future.