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

docs: add list with alternative implementations #169

Merged
merged 1 commit into from
Jul 6, 2023

Conversation

mnm678
Copy link
Collaborator

@mnm678 mnm678 commented Oct 21, 2021

Solves #128

It might be nice to clarify what implementations there are in other languages, but I couldn't find an active list of all the TUF implementations and don't think that this is the place to maintain that.

@mnm678 mnm678 force-pushed the notary-comparison branch 2 times, most recently from 6115317 to 975ad69 Compare October 21, 2021 22:45
@coveralls
Copy link

coveralls commented Oct 21, 2021

Pull Request Test Coverage Report for Build 1531746689

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 233 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+0.5%) to 68.352%

Files with Coverage Reduction New Missed Lines %
errors.go 9 0%
local_store.go 56 75.66%
repo.go 168 73.89%
Totals Coverage Status
Change from base Build 1356393585: 0.5%
Covered Lines: 2041
Relevant Lines: 2986

💛 - Coveralls

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@trishankatdatadog
Copy link
Member

@joshuagl can you please 👀 and ✅ ?

@joshuagl
Copy link
Member

joshuagl commented Dec 2, 2021

Will take a look next week

@rdimitrov
Copy link
Contributor

It might be nice to clarify what implementations there are in other languages, but I couldn't find an active list of all the TUF implementations and don't think that this is the place to maintain that.

I also think there's a need for such a list 👍

There's already an issue for the TUF website to have a list of some of the most meaningful implementations - theupdateframework/theupdateframework.io#27, so instead of having a list here maybe, it would be better to update the website and just refer to it? For example, a list of projects followed by a small description and maybe comparison info against some of the other similar projects)

@trishankatdatadog
Copy link
Member

Can we either merge/update/close this? Thanks!

rdimitrov
rdimitrov previously approved these changes Jul 3, 2023
@rdimitrov rdimitrov changed the title Notary comparison docs: add list with alternative implementations Jul 3, 2023
README.md Outdated Show resolved Hide resolved
@rdimitrov
Copy link
Contributor

@trishankatdatadog - hey, it seems that the DCO bot is complaining about one of your commits, so would you sign it and force push?

Another option is for @mnm678 or someone else (I can do it too if it's okay) to do an interactive rebase and squash all commits into a single signed one which hopefully will satisfy our bot.

explain difference between notary and go-tuf

Co-authored-by: Trishank Karthik Kuppusamy <trishank.kuppusamy@datadoghq.com>
Signed-off-by: Marina Moore <mnm678@users.noreply.github.com>
Signed-off-by: Marina Moore <mnm678@gmail.com>
@mnm678 mnm678 merged commit 4b9fd32 into theupdateframework:master Jul 6, 2023
25 checks passed
@mnm678 mnm678 deleted the notary-comparison branch July 6, 2023 22:28
@rdimitrov rdimitrov mentioned this pull request Jul 7, 2023
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.

None yet

5 participants