-
Notifications
You must be signed in to change notification settings - Fork 8
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
Directory structure rationalization #43
Comments
Perhaps we can move I hear from @dc-mak that the CN/Cerberus repo split is underway and will hopefully be complete in the not too distant future. |
That would make sense. It might lead to a bit of duplication, though,
since I guess we'll want to have many of the same (larger) examples in both
places...
…On Mon, Jul 15, 2024 at 4:24 PM Cole Schlesinger ***@***.***> wrote:
Perhaps we can move example-archive into the CN repo, once it's split
from the Cerberus repo? As remarked in the past, the example archive is
less about teaching CN to new users and more for characterizing CN
capabilities for us (tool developers) and for regression testing.
I hear from @dc-mak
<https://urldefense.com/v3/__https://github.com/dc-mak__;!!IBzWLUs!UvVlT0Iw0Tvg-TCe7AIvRd9Etqg3LGSjJJkbD_v4l7DSNQ60iLPTJayaeYzXVZDKhUL-DafEBiWHE2Vyr94vUTrRZ3mM$>
that the CN/Cerberus repo split is underway and will hopefully be complete
in the not too distant future.
—
Reply to this email directly, view it on GitHub
<https://urldefense.com/v3/__https://github.com/rems-project/cn-tutorial/issues/43*issuecomment-2229336053__;Iw!!IBzWLUs!UvVlT0Iw0Tvg-TCe7AIvRd9Etqg3LGSjJJkbD_v4l7DSNQ60iLPTJayaeYzXVZDKhUL-DafEBiWHE2Vyr94vUUhEpYdF$>,
or unsubscribe
<https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ABVQQCZOFY3SROBWSPJZ5ETZMQVW7AVCNFSM6AAAAABK5EEXUSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRZGMZTMMBVGM__;!!IBzWLUs!UvVlT0Iw0Tvg-TCe7AIvRd9Etqg3LGSjJJkbD_v4l7DSNQ60iLPTJayaeYzXVZDKhUL-DafEBiWHE2Vyr94vUTkYhzKe$>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I think some duplication is OK? I don't think it's a problem if the tutorial examples change independently of the example archive (e.g. improving readability, adding comments, etc.), and any structural changes we need to make, like syntax changes, will be more painful due to the number of examples in the archive rather than the relatively small bit of duplication. |
Agreed. The advantages of splitting are more than the slight pain of
duplication.
…On Mon, Jul 15, 2024 at 4:56 PM Cole Schlesinger ***@***.***> wrote:
I think some duplication is OK? I don't think it's a problem if the
tutorial examples change independently of the example archive (e.g.
improving readability, adding comments, etc.), and any structural changes
we need to make, like syntax changes, will be more painful due to the
number of examples in the archive rather than the relatively small bit of
duplication.
—
Reply to this email directly, view it on GitHub
<https://urldefense.com/v3/__https://github.com/rems-project/cn-tutorial/issues/43*issuecomment-2229418080__;Iw!!IBzWLUs!TTN0xEqZZn9quCs7SrvwXoxbPKwneBPYAh0O1qNflc9hU_wpRZH6VJePzUb84XkeJ8ETBA10--CIFNuc6Dh160CjTgTd$>,
or unsubscribe
<https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ABVQQC2MADFID76RNJOCARTZMQZQRAVCNFSM6AAAAABK5EEXUSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRZGQYTQMBYGA__;!!IBzWLUs!TTN0xEqZZn9quCs7SrvwXoxbPKwneBPYAh0O1qNflc9hU_wpRZH6VJePzUb84XkeJ8ETBA10--CIFNuc6Dh167hlfhLU$>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Agree. We should probably colocate |
That's what I was thinking too: having everything, including the tutorial, in one place will make the CI setup smoother. |
We should reorganize the directory structure to be more reasonable. Currently it looks like this:
Some anomalies:
example-archive
folder is buried, and we might want to lift it to the top levelsrc/examples
but by folder insrc/example-archive/
@bcpierce00
The text was updated successfully, but these errors were encountered: