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

Update github workflows #1178

Merged
merged 7 commits into from
Jan 20, 2025
Merged

Update github workflows #1178

merged 7 commits into from
Jan 20, 2025

Conversation

diksipav
Copy link
Contributor

@diksipav diksipav commented Jan 17, 2025

CLI still doesn't recognise [gel] inside gel.toml. So I updated our gel.toml files inside test folders to use egdedb.

Do we want to run some CI tests on Windows machines? All tests we have run on ubuntu.

At the end I reverted tests to use edgedb for the CLI (edgedb migrate etc...) in order to use the current setup-edgedb action.

@diksipav diksipav force-pushed the update-workflows branch 3 times, most recently from 404aed2 to e44ca17 Compare January 17, 2025 11:14
@diksipav diksipav force-pushed the update-workflows branch 2 times, most recently from 5bdac32 to 0f75ccb Compare January 20, 2025 14:34
@diksipav diksipav force-pushed the update-workflows branch 4 times, most recently from 7c68cbb to 8794924 Compare January 20, 2025 15:32
@diksipav diksipav added the gel related to renaming to gel label Jan 20, 2025
@diksipav diksipav merged commit 78c374a into gel Jan 20, 2025
9 checks passed
@diksipav diksipav deleted the update-workflows branch January 20, 2025 18:19
diksipav added a commit that referenced this pull request Jan 23, 2025
CLI still doesn't recognise [gel] inside gel.toml. So I updated our
`gel.toml` files inside test folders to use `egdedb`.

Do we want to run some CI tests on Windows machines? All tests we have
run on ubuntu.

At the end I reverted tests to use `edgedb` for the CLI (`edgedb
migrate` etc...) in order to use the current `setup-edgedb` action.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gel related to renaming to gel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant