Skip to content

Commit df94d69

Browse files
committed
Update FIXME message
1 parent ed242b6 commit df94d69

File tree

3 files changed

+11
-8
lines changed

3 files changed

+11
-8
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
"devDependencies": {
2727
"@iarna/toml": "^2.2.5",
2828
"@codama/renderers-js": "^1.0.0",
29-
"@codama/renderers-rust": "^1.0.0",
29+
"@codama/renderers-rust": "^1.0.1",
3030
"codama": "^1.0.0",
3131
"typescript": "^5.5.2",
3232
"zx": "^7.2.3"

pnpm-lock.yaml

Lines changed: 5 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

scripts/generate-clients.mjs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,10 @@ codama.accept(
1818
})
1919
);
2020

21-
// FIXME(loris): Temporary fix until the Rust renderer fixes the missing semicolon.
21+
// FIXME: Currently, the Rust renderer fails to generate a `ShortVec` because
22+
// it requires extra traits such as `BorshSerialize` that are not implemented
23+
// on the `ShortVec` type. To work around this issue, we remove the `configKeys`
24+
// type from Rust code generation and use a hooked type for it instead.
2225
codama.update(deleteNodesVisitor(['[definedTypeNode]configKeys']));
2326

2427
// Render Rust.
@@ -29,7 +32,7 @@ codama.accept(
2932
crateFolder: rustClient,
3033
toolchain: getToolchainArgument('format'),
3134

32-
// FIXME(loris): Temporary fix until the Rust renderer fixes the missing semicolon.
35+
// FIXME: The second part of the workaround for the `ShortVec` issue mentioned above.
3336
linkOverrides: {
3437
definedTypes: { configKeys: 'hooked' },
3538
},

0 commit comments

Comments
 (0)