Skip to content
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

Issue 1118 GitHub actions #1135

Merged
merged 11 commits into from
Nov 16, 2024
Merged

Issue 1118 GitHub actions #1135

merged 11 commits into from
Nov 16, 2024

Conversation

akaiap
Copy link
Collaborator

@akaiap akaiap commented Oct 30, 2024

This pull request will allow testing (previously Travis /CI) to be shown through Github Actions instead.

@dondi dondi changed the base branch from master to beta October 30, 2024 20:18
Copy link
Owner

@dondi dondi left a comment

Choose a reason for hiding this comment

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

Looking good—we fixed the merge target and that eliminated all of the extra diffs; plus a couple of comments, particularly the one with the line break which may fix the (current) error

Comment on lines 27 to 28
sudo apt-get install -y libcairo2-dev libpango1.0-dev
libjpeg-dev libgif-dev librsvg2-dev
Copy link
Owner

Choose a reason for hiding this comment

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

I think this should stay as one line; the error hints that GitHub’s environment is trying to perform the libjpeg-dev line as a command, and that would be due to its being on another line


strategy:
matrix:
node-version: [18.x, 20.x, 22.x]
Copy link
Owner

Choose a reason for hiding this comment

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

Per our meeting, to simplify things for now, let’s reduce this to just one version until things are more settled

@akaiap
Copy link
Collaborator Author

akaiap commented Nov 13, 2024

The branch has now passed all the checks and CI testing has been successfully integrated into GitHub Actions!

Copy link
Owner

@dondi dondi left a comment

Choose a reason for hiding this comment

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

LGTM

@dondi dondi merged commit cd7d8b3 into beta Nov 16, 2024
1 check passed
@dondi dondi deleted the issue-1118-github-actions branch November 16, 2024 02:39
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