-
-
Notifications
You must be signed in to change notification settings - Fork 95
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
[Core] Stop the DSA PR #61
Comments
We should make sub-sections in a way such that it's more useful for implementation, |
@sreekar9601 great idea @mkubdev I think continuing won't be a bad idea. Also, I think we have to give a thought to writing algos in Yul and Huff since they're more suited for mathematical operations. @iSumitBanik and @sreekar9601 thanks for your contributions, its nice to not be alone anymore. |
Agreed with this point. Writing in Yul/Huff would let us contribute some heavily gas optimized contracts ;) |
@mkubdev - I would be interested in contributing for non DSA, I am learning solidity by example. Should I add some simple contracts which I am trying for learning the language features? |
@priyda - sure! :) |
How do you think packaging this examples? |
@mkubdev - I have raised a pull request #77, can you please review that once? If looks good. I have more to add for your review :) |
Sure, i'll review it as soon as i'm available. :D |
I added a few comments from my side. |
Hello 👋🏻
This repos is taking a wrong direction. Solidity language isn't suited for heavy mathematics computing.
I think we need to remove the list that looks like DSA, and restructure the repos to list Solidity use cases instead.
The text was updated successfully, but these errors were encountered: