Skip to content
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 README.md #6

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

simonletort-da
Copy link

added description for Daml and Canton

added description for Daml and Canton
@simonletort-da
Copy link
Author

@garyverhaegen-da can you review or assign someone to review? thanks.

@garyverhaegen-da
Copy link

@garyverhaegen-da can you review or assign someone to review? thanks.

I'm sorry I don't have any context here. What is this repo, and why am I involved?

@simonletort-da
Copy link
Author

It's the DA orga readme. I made some basic updates. Not sure who has merge rights.

Comment on lines +9 to +11
[Daml](https://github.com/digital-asset/daml) is an open-source smart contract language for building future-proof distributed applications on a safe, privacy-aware runtime.

[Canton](https://github.com/digital-asset/canton) is a Daml ledger interoperability protocol.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not an accurate description of the repositories. I think I'd go more for something like:

Suggested change
[Daml](https://github.com/digital-asset/daml) is an open-source smart contract language for building future-proof distributed applications on a safe, privacy-aware runtime.
[Canton](https://github.com/digital-asset/canton) is a Daml ledger interoperability protocol.
Our primary open-source offering is Daml Open-Source, which is composed of the following components:
* [list of components]
These components are distributed across the following repositories:
* [digital-asset/daml](https://github.com/digital-asset/daml) contains the code for [compiler, IDE, HTTP, Triggers, Script, etc.]
* [digital-asset/canton](https://github.com/digital-asset/canton) contains the code for [participant, sequencer, ...]
For historical reasons, the fully assembled Daml Open-Source distribution is hosted as GitHub Releases on the [digital-asset/daml] repo. Note that this means releases on that repo include artifacts produced from other repositories.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I only copy/pasted the current description from the official Daml and Canton readme.
I'll work with marketing to cleanup these readme and then update the DA one accordingly.

@garyverhaegen-da
Copy link

It's the DA orga readme. I made some basic updates. Not sure who has merge rights.

I didn't know orgs had readmes 🤷‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants