sui
sui copied to clipboard
Corrects a directory name. Adds parent directories
💳 Wallet Extension has been built, you can download the packaged extension here: https://github.com/MystenLabs/sui/actions/runs/3321657872#artifacts
I don't agree with this change. The linked text shows the name of the component, it doesn't need to also include part of the path
@randall-Mysten Fair point. In which case, do you think it's useful to update the sentence introducing the enumeration, i.e.:
You can start exploring Sui's source code by looking into the following primary directories: to clarify how "primary directories" should be interpreted by the reader?
Is the intention to call out the primary directories, or list the key components and link to the folder that contains them? As a user, am I looking for a primary folder, or source code for a binary or app? What if it listed these things, linked to the folder as it does now, and displayed the path inline so people that want to navigate through the repo can easily find them?
Please reopen if this is still a relevant issue. Closing stale PRs