Skip to content

Commit

Permalink
feat: use json as manifest constants instead of toml (#922)
Browse files Browse the repository at this point in the history
* feat: use json as manifest constants instead of toml

Currently, we parse the manifest file as TOML. However, we will use JSON
for the genesis format. For improved consistency, it is desirable to
have the constants manifest file with the same format.

* fix CI lints

* use parent as argument for error reporting

* refactor gas config parse to return declaration

* fix ci lints
  • Loading branch information
vlopes11 authored and dubbelosix committed Sep 29, 2023
1 parent 8221543 commit 5af3474
Showing 6 changed files with 220 additions and 101 deletions.
3 changes: 3 additions & 0 deletions constants.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
{
"comment": "Sovereign SDK constants"
}
1 change: 0 additions & 1 deletion constants.toml

This file was deleted.

3 changes: 1 addition & 2 deletions module-system/sov-modules-macros/Cargo.toml
Original file line number Diff line number Diff line change
@@ -23,7 +23,6 @@ path = "tests/all_tests.rs"
clap = { workspace = true }
jsonrpsee = { workspace = true, features = ["macros", "http-client", "server"] }
serde = { workspace = true }
serde_json = { workspace = true }
tempfile = { workspace = true }
trybuild = "1.0"

@@ -39,8 +38,8 @@ jsonrpsee = { workspace = true, features = ["http-client", "server"], optional =
proc-macro2 = "1.0"
quote = "1.0"
schemars = { workspace = true }
serde_json = { workspace = true }
syn = { version = "1.0", features = ["full"] }
toml = "0.8"

[features]
default = []
Loading

0 comments on commit 5af3474

Please sign in to comment.