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
update aptos.toml #1227
base: master
Are you sure you want to change the base?
update aptos.toml #1227
Conversation
Hey @jubos , could you please help to review about PR? Also I have a question about the sub-ecosystem developer/commit calculation? For example, pancakeswap is the sub ecosystem for aptos. But when EC count commits or developers for aptos, did EC count all commits from pancakeswap repos? Since pancakeswap is the sub-ecosystem for many chains, I am a little bit confuse about the calculation logic. Thanks! |
Hey @jubos , could you please help to review this PR? Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hhuang728, thank you for the PR.
As of now, this casts a bit too wide of a net. There are a lot of developer forks of aptos-core with no changes, so we tend not to add those because they can pollute the taxonomy.
Can you scope this change and add some of the easier ones (e.g. repos that have aptos in the name or have many mentions of Aptos) and then do a separate PR for some of the sub ecosystem additions like Wormhole or the Arweave related repos that may or may not be Aptos related?
Hi @jubos , thanks for your reply!
|
Yes, unless they have significant commits that are different from aptos-core.
You can do a grep of aptos in the codebase or in the README.md. Some of the repos that might be pulling in a transitive dependency on a project that has aptos mentioned somewhere would have a very tenuous connection to Aptos and probably shouldn't be included.
Those should be added into Wormhole and Wormhole can be a sub ecosystem of Aptos instead of adding them directly to the aptos.toml file. |
Hi @jubos, Thanks for providing detail feedback. I just updated the repo list based on above discussion. However there is Error: Process completed with exit code 2 in the validation check. Could you please help to take a look? Thanks! |
Hi @jubos ,got it. just submitted based on all previous feedback. Please help to review. Thank you! |
Hey @jubos , thanks for all of your previous help! |
add new repo without depulicated repo in origianl toml file