Skip to content

Remove travis, add github workflow - #211

Merged
michalz-zoom merged 1 commit into
masterfrom
michal/add-ci
Sep 22, 2026
Merged

michalz-zoom merged 1 commit into
masterfrom
michal/add-ci

Conversation

@michalz-zoom

Copy link
Copy Markdown
Contributor

@zoom-ua let's add github CI here.

do we need to set anything in the repository settings? I don't have access to that

@zoom-ua zoom-ua left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chrisnojima @chrisnojima-zoom can you enable CI here?

Comment thread .github/workflows/ci.yml Outdated

strategy:
matrix:
node-version: [22, 24]

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
node-version: [22, 24]
node-version: [24, 26]

Comment thread .github/workflows/ci.yml Outdated

steps:
- name: Checkout code
uses: actions/checkout@v4

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

v7 is the latest here and below, why not use those?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kk i just copied this over from kbpgp, ill change there as well

@michalz-zoom michalz-zoom changed the title Remove traivs, add github workflow Remove travis, add github workflow Sep 22, 2026
@michalz-zoom

Copy link
Copy Markdown
Contributor Author

ok this seems to work

@michalz-zoom
michalz-zoom merged commit 5155639 into master Sep 22, 2026
3 checks passed
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