-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #10 from stackql/feature/updates
Feature/updates
- Loading branch information
Showing
7 changed files
with
179 additions
and
27 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
## Description | ||
|
||
Please include a summary of the changes and the related issue(s). Please also include relevant motivation and context. | ||
|
||
Fixes # | ||
|
||
## Checklist | ||
|
||
Please make sure that the following criteria are met: | ||
|
||
- [ ] The PR title is descriptive. | ||
- [ ] Include documentation for example queries (comments or markdown in notebooks) | ||
- [ ] I have ⭐'ed the [stackql](https://github.com/stackql/stackql) and [stackql-jupyter-demo](https://github.com/stackql/stackql-jupyter-demo) repos. | ||
|
||
## Additional Notes | ||
|
||
Add any additional information or context that might help the reviewers. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,7 +3,6 @@ name: Build and Publish | |
on: | ||
push: | ||
branches: [main] | ||
pull_request: | ||
|
||
jobs: | ||
build-and-push-docker-image: | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,45 @@ | ||
name: Check if PR author has starred the repository | ||
on: | ||
pull_request: | ||
types: [opened, synchronize, reopened] | ||
|
||
jobs: | ||
check-starred: | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v4.1.7 | ||
|
||
- name: Get PR author username | ||
id: get-author | ||
run: echo "username=${{ github.event.pull_request.user.login }}" >> $GITHUB_ENV | ||
|
||
- name: Pull github provider | ||
uses: stackql/stackql-exec@v2.2.1 | ||
with: | ||
is_command: 'true' | ||
query: "REGISTRY PULL github;" | ||
|
||
- name: Run stackql query | ||
id: check-star | ||
uses: stackql/stackql-assert@v2.2.1 | ||
with: | ||
test_query: | | ||
SELECT repo, count(*) as has_starred | ||
FROM github.activity.repo_stargazers | ||
WHERE owner = 'stackql' and repo in ('stackql', 'stackql-jupyter-demo') and login = '${{ env.username }}' | ||
GROUP BY repo; | ||
expected_results_str: '[{"has_starred":"1","repo":"stackql"},{"has_starred":"1","repo":"stackql-jupyter-demo"}]' | ||
continue-on-error: true | ||
|
||
- name: Check if starred | ||
if: always() # Ensures this runs regardless of check-star outcome | ||
run: | | ||
if [ "${{ steps.check-star.outcome }}" = "success" ]; then | ||
echo "::notice::Thanks for your support!" | ||
else | ||
echo "::error::It seems you haven't starred the StackQL repositories. Please star the following repos before proceeding: https://github.com/stackql/stackql-jupyter-demo (this repo) and https://github.com/stackql/stackql (our core repo)" | ||
exit 1 | ||
fi | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,2 @@ | ||
#!/bin/sh | ||
/srv/stackql/stackql --version | ||
echo "starting stackql server..." | ||
nohup /srv/stackql/stackql --execution.concurrency.limit=-1 --dataflow.components.max=50 --dataflow.dependency.max=50 --http.response.pageLimit=-1 --pgsrv.port=5466 srv & | ||
echo "stackql server started" | ||
start-notebook.sh --NotebookApp.token='' | ||
sh /scripts/start-server.sh && start-notebook.sh --NotebookApp.token='' |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
#!/bin/sh | ||
/srv/stackql/stackql --version | ||
echo "starting stackql server..." | ||
nohup /srv/stackql/stackql --execution.concurrency.limit=-1 --dataflow.components.max=50 --dataflow.dependency.max=50 --http.response.pageLimit=-1 --pgsrv.port=5466 srv & | ||
echo "stackql server started" |