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.

Sign up for GitHub

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

Jump to bottom

ci: add more triggers #1440

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

ci: add more triggers #1440

wants to merge 3 commits into from
+190 −317

Conversation

Yaiba
Copy link
Contributor

@Yaiba Yaiba commented Mar 3, 2025 •
edited
Loading

This pr changes multiple places in CI

  • fix kgw test with kwil-js , a manual triggerred test https://github.com/kwilteam/kwil-db/actions/runs/13662100865
  • add manual test trigger for integration test. NEW manual CI workflow won't show up in action page until it's in main branch. So I have to merge this first, then test
  • modify contrib/docker/compose/kwil/docker-compose.yml so we can configure a db owner

Yaiba
Yaiba commented Mar 4, 2025
View reviewed changes
README.md
@@ -130,7 +130,7 @@ configuration changes. Start it by running the following from the

```sh
cd contrib/docker/compose/kwil
docker compose up --build -d
KWIL_DB_OWNER=YOUR_ETH_ADDRESS docker compose up --build -d
Copy link
Contributor Author

Choose a reason for hiding this comment

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

So the db owner is configured

@Yaiba Yaiba force-pushed the ci/more-trigger branch 2 times, most recently from 4744ab3 to ecceef8 Compare March 4, 2025 20:12
@Yaiba Yaiba marked this pull request as ready for review March 4, 2025 20:26
@Yaiba Yaiba changed the title [WIP]ci: add more triggers ci: add more triggers Mar 4, 2025
@Yaiba Yaiba force-pushed the ci/more-trigger branch from ecceef8 to bd7e43b Compare March 4, 2025 20:47
@Yaiba Yaiba force-pushed the ci/more-trigger branch from bd7e43b to ce67e0e Compare March 4, 2025 23:07
@Yaiba Yaiba requested a review from jchappelow March 5, 2025 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jchappelow jchappelow Awaiting requested review from jchappelow

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Yaiba