Skip to content

Restore CI to working state#1007

Merged
corranwebster merged 16 commits into
mainfrom
fix/restore-ci
Mar 6, 2023
Merged

Restore CI to working state#1007
corranwebster merged 16 commits into
mainfrom
fix/restore-ci

Conversation

@corranwebster
Copy link
Copy Markdown
Contributor

@corranwebster corranwebster commented Jan 13, 2023

This gets CI back into a working state by:

Everything is passing: https://github.com/enthought/enable/actions/runs/4186677329

There is a warning about set-output, but I don't think it is us.

jwiggins
jwiggins previously approved these changes Feb 15, 2023
Copy link
Copy Markdown
Member

@jwiggins jwiggins left a comment

Choose a reason for hiding this comment

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

LGTM

@jwiggins jwiggins dismissed their stale review February 15, 2023 15:49

GitHub showed me the wrong diff. Still work to do here.

Copy link
Copy Markdown
Member

@jwiggins jwiggins left a comment

Choose a reason for hiding this comment

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

LGTM (for real this time)

@corranwebster corranwebster merged commit 71c3bb2 into main Mar 6, 2023
@corranwebster corranwebster deleted the fix/restore-ci branch March 6, 2023 09:46
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