-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Make necessary refactoring to enable no_std
support
#62
Comments
@mitchmindtree are you still planning on working on this? |
Ahh apologies, I intended to look into it as a way to get started but have been consumed by forc stuff :) I'll unassign myself in case anyone else would like to take this on! |
@digorithm are you still planning on working on this? 😂 |
To be honest, only if/when it's actually needed. There are so many more higher priority items that this one is way lower in priority. Of course its priority would change if it becomes a blocker to someone. Since we removed no_std support I haven't heard any requests for it, so I'm assuming it's not needed at the moment. |
It's causing circular dependency issues between fuel-core and fuel-indexer, and fuel-indexer is stuck on an outdated version of fuels-rs (v0.2) until this is resolved. We can potentially get someone from the client team to look into this if you don't have bandwidth. |
Closing this in favour of #317 as we've discovered |
I've opened #318. |
Due to many issues with the current way we were enabling
no_std
support onfuels-rs
, we had to revert and removeno_std
support for now. Previous versions will still work normally.However, we still want to enable
no_std
to some parts offuels-rs
that are needed asno_std
, such as the Fuel's indexer. This will require some major refactoring/reorganization in order to make it moreno_std
friendly.Some parts that need to be included as
no_std
:wasm_abigen
;Token
,Tokenizable
,ParamType
, and related dependencies.The text was updated successfully, but these errors were encountered: