ethereum-oasis-op / baseline-blips

Baseline Protocol Improvement Proposals (BLIPs) play a key role in properly proposing, developing, and implementing changes to the Baseline Protocol. This repo contains all BLIPs.
Creative Commons Zero v1.0 Universal
10 stars 5 forks source link

[BLIP-3] Example, BRI and Integration Folder Structure #5

Closed humbitious closed 1 year ago

humbitious commented 2 years ago

Example, BRI and Integration Folder Structure

The current folder structure on the Main branch is confusing specifically when it comes to BRI (reference implementations), which you currently have to discover in the Examples folder, which makes no sense. And actual examples, like the hello world battleship game, you would have to discover inside yet another examples folder inside BRI-1, which makes it seem like it is only an example for BRI-1. And then there are integrations, like SAP, Dynamics and Excel. Integration libraries, we argue, should a) be independent of any given BRI, and b) should be found at the top level of the repo folder structure, so that they are easy to find, and so that others are encouraged to contribute more integration libraries to more things over time.

author: John Wolpert (@humbitious)

This BLIP is a proposal for a simple set of folder changes. The only test of completion is to see the correct folder structure as described above and to test that the new folder structure does not break anything in the code.


Copyright Copyright and related rights waived via CC0-Universal.

(This template adapted from the EIP template at https://github.com/ethereum/EIPs/

skosito commented 2 years ago

@humbitious As discussed, I will pick this up

Therecanbeonlyone1969 commented 2 years ago

Proposal on Structure:

docs -> subfolders as required

reference-implementation -> subfolders one per reference implementations

examples -> subfolders one per example

.dependabot and .github should stay at the top.

@humbitious

GoldenBit0 commented 2 years ago

@skosito Please see above proposal as well - do you have an update on your work with this? TIA

skosito commented 2 years ago

Thanks for proposal @Therecanbeonlyone1969 , @GoldenBit0 I will have some updates until Monday.

Couple of questions regarding proposed structure:

-examples -> subfolders one per example If I understood correctly, currently there are no examples like hello world, battleship etc. that would be in this folder, so it would remain empty for now?

skosito commented 2 years ago

@humbitious @GoldenBit0 @Therecanbeonlyone1969 Hey, I pushed simple change here, please check if this makes sense on a folder structure level, it should be visible like this: https://github.com/skosito/baseline/tree/add-ref-implementations-folder Pretty much I renamed examples to referenced-implementations and then run bri-1 and bri-2 readme commands and checked if i get same results. I did not do any testing yet for remaining (deprecated radish34 and lib folder in bri-1) since they seem more involved for verification, but I will try verify as well - if you have some advice here, please let me know.

Then I cleaned up mentions of examples folder (mostly in urls) across codebase and replaced it with reference-implementations (I might revisit this and also leave examples folder in some .npmignore files since we will probably need it once we actually add examples folder again).

There is one more place that is referencing examples folder, in couple of json files in bri-2, but it seems like an local output of some tool (truffle?), in this format (this is Shield.json file in bri-2): "sourcePath": /Users/samuelstokes/repos/Web3Studio/baseline/examples/bri-2/contracts/contracts/privacy/Shield.sol. Do we need these types of files, since they seem connected to local build anyways?

Please review this and let me know your thoughts.

humbitious commented 2 years ago

Hey @skosito! Nice work! I took a look, and other than obviously needing to add a new /examples folder, looks right to me. Let's merge it. @bitwiseguy can comment on the json files in bri-2.

Let's be sure that Radish-34, if it isn't already, is solidly marked as deprecated...I think it is.)

skosito commented 2 years ago

@humbitious , thanks for checking it, I opened PR for this.

GoldenBit0 commented 2 years ago

Maintainers to review/approve PR 432- https://github.com/eea-oasis/baseline/pull/432

GoldenBit0 commented 2 years ago

12/13/21 Core Devs Session:

GoldenBit0 commented 2 years ago

1/10/22 Core Devs Session:

Outcome:

GoldenBit0 commented 2 years ago

5/2/22 Core Devs Call:

GoldenBit0 commented 2 years ago

6/13/22 Core Devs Call:

GoldenBit0 commented 2 years ago

6/27/22 Core Devs Call:

GoldenBit0 commented 2 years ago

@kthomas Hi Kyle, per the Core Devs call today, we are going to proceed with the folder changes. Please review if this will break any links or cause issues for you, and let us know by tomorrow.

Thanks!

kthomas commented 2 years ago

@kthomas Hi Kyle, per the Core Devs call today, we are going to proceed with the folder changes. Please review if this will break any links or cause issues for you, and let us know by tomorrow.

Thanks!

Thanks Sonal, I would like to see the PR and then I can comment...

GoldenBit0 commented 2 years ago

7/11/22 Core Devs:

GoldenBit0 commented 2 years ago

7/25/22 Core Devs:

GoldenBit0 commented 2 years ago

9/19/22 Core Devs:

GoldenBit0 commented 1 year ago

10/3/22 Core Devs:

GoldenBit0 commented 1 year ago

10/17/22 Core Devs:

GoldenBit0 commented 1 year ago

10/31/22 Core Devs:

GoldenBit0 commented 1 year ago

11/14/22: