Skip to content

Switch to GitHub Actions CI#79

Closed
XhmikosR wants to merge 1 commit into
mklabs:masterfrom
XhmikosR:ci
Closed

Switch to GitHub Actions CI#79
XhmikosR wants to merge 1 commit into
mklabs:masterfrom
XhmikosR:ci

Conversation

@XhmikosR
Copy link
Copy Markdown
Contributor

@XhmikosR XhmikosR commented Jan 23, 2022

Tests seem to fail for some reason on GitHub Actions.

Preview: https://github.com/XhmikosR/tabtab/actions?query=branch%3Aci

@XhmikosR
Copy link
Copy Markdown
Contributor Author

XhmikosR commented Jan 23, 2022

@erezrokah: do you have any ideas how to fix the CI?

I also made #80 which updates most deps, and reduces the deps from 6 to 3.

@erezrokah
Copy link
Copy Markdown

@erezrokah: do you have any ideas how to fix the CI?

Previous CI also does

- npm install -g npm@latest

Not sure how that works on older Node.js versions. See https://github.com/npm/cli/blob/44bbb4933a0fe35999f5628d4bfd8ebb15981763/package.json#L262

@XhmikosR
Copy link
Copy Markdown
Contributor Author

I don't think it's related to npm at all, if you see the errors it has something to do with GitHub Actions not loading/having the needed rights for the user folder. But I have no idea how to fix the errors :/

@XhmikosR XhmikosR closed this Mar 31, 2023
@XhmikosR XhmikosR deleted the ci branch March 31, 2023 11:48
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