-
Notifications
You must be signed in to change notification settings - Fork 0
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
Initial Development #1
Changes from 53 commits
35bb733
1005d12
bb036a5
aca71cf
c29dcea
c2ac93d
5d4ed38
64f65f2
69d818e
a472f0b
d820f02
dfa1069
36fa261
297cf71
43df208
a972d41
2085f10
3fa48e6
828cda1
9265d71
92052c8
701386c
d77c744
1f87290
bbb682f
cdbcc96
30e9d75
907993f
b6fd292
f0ba829
f10ea03
49f83b5
8a07cb7
fac3a4c
9923657
621fff1
cb1ef4f
6a7f87e
b68f846
ded8220
e356b47
3ceffff
5d23331
e7cdc42
218d520
cddc83d
8e40bd2
ac97fd4
cffbea0
23b888d
0edc166
fbb431d
ede07c9
c17c3da
1850044
1d63b55
a60b6c2
8e8a903
d9959a1
07d9b80
3ccc462
cbc796a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
version: 2 | ||
updates: | ||
# This is mostly for actions/checkout | ||
- package-ecosystem: "github-actions" | ||
directory: "/" | ||
schedule: | ||
interval: "weekly" | ||
target-branch: "main" | ||
|
||
# This is actually for cargo crates | ||
- package-ecosystem: cargo | ||
directory: "/" | ||
schedule: | ||
interval: "weekly" | ||
target-branch: "main" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
name: Linux Build + Tests | ||
|
||
on: | ||
push: | ||
branches: [ "main" ] | ||
|
||
env: | ||
CARGO_TERM_COLOR: always | ||
|
||
jobs: | ||
build-linux: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Build | ||
run: cargo build | ||
- uses: taiki-e/install-action@nextest | ||
- name: Populate test graph | ||
run: pip install falkordb && ./resources/populate_graph.py | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Consider using a virtual environment for Python dependencies to avoid potential conflicts with system packages. Tip Codebase Verification The
Analysis chainEnsure that the Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Check error handling in populate_graph.py script.
cat ./resources/populate_graph.py | grep -i "try"
Length of output: 49 Script: #!/bin/bash
# Description: Check for error handling mechanisms in populate_graph.py script.
grep -Ei "try|except|raise|logging" ./resources/populate_graph.py
Length of output: 67 Script: #!/bin/bash
# Description: Display the entire content of populate_graph.py script for review.
cat ./resources/populate_graph.py
Length of output: 1924 |
||
- name: Test | ||
run: cargo nextest run --all | ||
services: | ||
falkordb: | ||
image: falkordb/falkordb:edge | ||
ports: | ||
- 6379:6379 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
name: Code Coverage | ||
|
||
on: | ||
push: | ||
branches: [ "main" ] | ||
|
||
env: | ||
CARGO_TERM_COLOR: always | ||
|
||
jobs: | ||
coverage: | ||
runs-on: linux-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Populate test graph | ||
run: pip install falkordb && ./resources/populate_graph.py | ||
- uses: taiki-e/install-action@cargo-llvm-cov | ||
- uses: taiki-e/install-action@nextest | ||
- name: Generate Code Coverage | ||
run: cargo llvm-cov nextest --all --codecov --output-path codecov.json | ||
- name: Upload coverage reports to Codecov | ||
uses: codecov/codecov-action@v4 | ||
with: | ||
token: ${{ secrets.CODECOV_TOKEN }} | ||
files: codecov.json | ||
fail_ci_if_error: true | ||
services: | ||
falkordb: | ||
image: falkordb/falkordb:edge | ||
ports: | ||
- 6379:6379 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
name: Documentation | ||
|
||
on: | ||
push: | ||
branches: [ "main" ] | ||
|
||
env: | ||
CARGO_TERM_COLOR: always | ||
|
||
jobs: | ||
doc: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Doc | ||
run: cargo doc --all |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
name: Mandatory Pull Request Checks | ||
|
||
on: | ||
pull_request: | ||
branches: [ "main" ] | ||
|
||
env: | ||
CARGO_TERM_COLOR: always | ||
|
||
jobs: | ||
check-clippy: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Check Clippy | ||
run: cargo clippy --all | ||
|
||
check-deny: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- uses: EmbarkStudios/cargo-deny-action@v1 | ||
|
||
check-doc: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Doc | ||
run: cargo doc --all | ||
|
||
check-fmt: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Check Rustfmt | ||
run: cargo fmt --all --check | ||
|
||
build-linux: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Build | ||
run: cargo build | ||
- uses: taiki-e/install-action@nextest | ||
- name: Populate test graph | ||
run: pip install falkordb && ./resources/populate_graph.py | ||
- name: Test | ||
run: cargo nextest run --all | ||
services: | ||
falkordb: | ||
image: falkordb/falkordb:edge | ||
ports: | ||
- 6379:6379 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,83 @@ | ||
name: Publish on crates.io | ||
|
||
on: | ||
push: | ||
tags: [ '*' ] | ||
|
||
env: | ||
CARGO_TERM_COLOR: always | ||
|
||
jobs: | ||
# Ensure formatting | ||
fmt: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Check Rustfmt | ||
run: cargo fmt --all --check | ||
|
||
# Make sure no unwanted licenses or yanked crates have slipped in | ||
deny: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- uses: EmbarkStudios/cargo-deny-action@v1 | ||
|
||
# Make sure the release's build works | ||
build: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Build | ||
run: cargo build | ||
- uses: taiki-e/install-action@nextest | ||
- name: Populate test graph | ||
run: pip install falkordb && ./resources/populate_graph.py | ||
- name: Test | ||
run: cargo nextest run --all | ||
services: | ||
falkordb: | ||
image: falkordb/falkordb:edge | ||
ports: | ||
- 6379:6379 | ||
|
||
# Ensure no clippy warnings | ||
clippy: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Check Clippy | ||
run: cargo clippy --all | ||
|
||
# Make sure the release's docs are full | ||
doc: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Doc | ||
run: cargo doc --all | ||
|
||
# Actually publish to crates.io | ||
crates-io: | ||
needs: | ||
- fmt | ||
- deny | ||
- build | ||
- clippy | ||
- doc | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Install Cargo Release | ||
run: cargo install cargo-release | ||
- name: Login | ||
run: cargo login ${{ secrets.CRATES_IO_API_TOKEN }} | ||
- name: Publish | ||
run: |- | ||
cargo release \ | ||
publish \ | ||
--all-features \ | ||
--allow-branch HEAD \ | ||
--no-confirm \ | ||
--no-verify \ | ||
--execute |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,6 @@ | ||
target/ | ||
/target | ||
.idea/ | ||
.vscode/ | ||
.vs/ | ||
/codecov.json | ||
/dump.rdb |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider using a virtual environment for Python dependencies to avoid potential conflicts with system packages.
Tip
Codebase Verification
The
populate_graph.py
script does not contain any explicit error handling mechanisms such as try-except blocks or logging. This could lead to the build failing unexpectedly if any errors occur during its execution.Analysis chain
Ensure that the
populate_graph.py
script handles errors gracefully to prevent the build from failing unexpectedly.Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 49
Script:
Length of output: 67
Script:
Length of output: 1924