Skip to content
This repository has been archived by the owner on Dec 23, 2024. It is now read-only.

Readme and License #64

Merged
merged 3 commits into from
Jan 24, 2024
Merged

Readme and License #64

merged 3 commits into from
Jan 24, 2024

Conversation

youben11
Copy link
Member

No description provided.

Copy link
Contributor

@aquint-zama aquint-zama left a comment

Choose a reason for hiding this comment

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

minor remarks

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
Integration.md Outdated

This document is a guide listing detailed steps to integrate [fhevm-go](https://github.com/zama-ai/fhevm-go) into [go-ethereum](https://github.com/ethereum/go-ethereum) or any other implementations that follows the same architecture.

**Note:** This document is based on go-ethereum v1.13.5
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
**Note:** This document is based on go-ethereum v1.13.5
> **Note**
> This document is based on go-ethereum v1.13.5

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, I could use them in unindented notes, but it didn't work for indented ones so I left them as is

Integration.md Show resolved Hide resolved
@youben11 youben11 force-pushed the readme branch 2 times, most recently from 78e6a7c to 22a000d Compare January 18, 2024 08:59
Integration.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved

## Documentation

Full, comprehensive documentation is available at [https://docs.zama.ai/fhevm](https://docs.zama.ai/fhevm) (still nothing about fhevm-go however).
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we remove the note about fhevm-go missing from fhevm doc?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, but not until we know what to do about the docs

Copy link
Contributor

@dartdart26 dartdart26 left a comment

Choose a reason for hiding this comment

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

Looks great! Added some minor suggestions.

@youben11 youben11 merged commit 3dd1995 into main Jan 24, 2024
1 check passed
@youben11 youben11 deleted the readme branch January 24, 2024 17:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants