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

feat: aptos da client #9

Merged
merged 7 commits into from
Aug 29, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .cargo/config.toml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
[build]
rustflags = ["--cfg", "tokio_unstable"]
7 changes: 7 additions & 0 deletions .env.example
Original file line number Diff line number Diff line change
Expand Up @@ -33,3 +33,10 @@ SQS_JOB_VERIFICATION_QUEUE_URL=
# S3
AWS_S3_BUCKET_NAME=
AWS_S3_BUCKET_REGION=

#APTOS
APTOS_NODE_URL=
APTOS_PRIVATE_KEY=
APTOS_MODULE_ADDRESS=
APTOS_CHAIN_ID=
APTOS_CRS_PATH=
22 changes: 22 additions & 0 deletions .github/auto_assign.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
# Set to true to add reviewers to pull requests
addReviewers: true

# Set to true to add assignees to pull requests
addAssignees: true

# A list of reviewers to be added to pull requests (GitHub user name)
reviewers:
- Draply
- 0x5ea000000
- hduoc2003
- zk-steve
- Tranduy1dol
- VanhGer

# A list of keywords to be skipped the process that add reviewers if pull requests include it
skipKeywords:
- wip

# A number of reviewers added to the pull request
# Set 0 to add all the reviewers (default: 0)
numberOfReviewers: 0
6 changes: 6 additions & 0 deletions .github/dependabot.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
version: 2
updates:
- package-ecosystem: "gitsubmodule"
directory: "/"
schedule:
interval: "daily"
13 changes: 13 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
# Check List

- [ ] Don't forget to squash commits into meaningful chunks before merging
- [ ] Check every test passed.
- [ ] Did you split imports into std, third parties, and custom parts?
- [ ] Format your code.
- [ ] cargo fmt
- [ ] cargo clippy
- [ ] `taplo fmt --config taplo/taplo.toml`
- [ ] Check unwrap, expect().
- [ ] Check clone().
- [ ] Check your commit messages.
- [ ] Have you added meaningful comments.
1 change: 1 addition & 0 deletions .github/workflows/coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ jobs:

- name: Run llvm-cov
run: |
sudo apt install build-essential cmake pkg-config libudev-dev
cargo llvm-cov nextest --release --lcov --output-path lcov.info --test-threads=1

- name: Coveralls
Expand Down
7 changes: 4 additions & 3 deletions .github/workflows/linters-cargo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ jobs:
- uses: Swatinem/rust-cache@v2
- name: Format and clippy
run: |
cargo fmt -- --check
cargo clippy --no-deps -- -D warnings
cargo clippy --tests --no-deps -- -D warnings
sudo apt install build-essential cmake pkg-config libudev-dev
Tranduy1dol marked this conversation as resolved.
Show resolved Hide resolved
RUSTFLAGS="--cfg tokio_unstable" cargo fmt -- --check
RUSTFLAGS="--cfg tokio_unstable" cargo clippy --no-deps -- -D warnings
RUSTFLAGS="--cfg tokio_unstable" cargo clippy --tests --no-deps -- -D warnings
4 changes: 2 additions & 2 deletions .github/workflows/pull-request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@ name: Workflow - Pull Request
on:
workflow_dispatch:
pull_request:
branches: [main]
branches: [aptos]
push:
branches: [main]
branches: [aptos]
Comment on lines +7 to +9

Choose a reason for hiding this comment

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

remove


jobs:
linters:
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/rust-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,5 @@ jobs:

- name: Build the project
run: |
cargo build --release --workspace
sudo apt install build-essential cmake pkg-config libudev-dev
RUSTFLAGS="--cfg tokio_unstable" cargo build --release --workspace
3 changes: 3 additions & 0 deletions .gitmodules
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
[submodule "ionia"]
path = ionia
url = [email protected]:sota-zk-labs/ionia.git
4 changes: 3 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,18 +22,20 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
- Tests for DA job.
- Added generalized errors for Jobs : JobError.
- Database tests
- Aptos DA client, Aptos-compatible workflow.
Tranduy1dol marked this conversation as resolved.
Show resolved Hide resolved

## Changed

- GitHub's coverage CI yml file for localstack and db testing.
- Orchestrator :Moved TestConfigBuilder to `config.rs` in tests folder.
- `.env` file requires two more variables which are queue urls for processing
and verification.
- Shifted Unit tests to test folder for DA job.

## Removed

- `fetch_from_test` argument

## Fixed

-
- Fixed state update worker logic as per the new implementation.
Loading
Loading