Correct README to reflect the defork from mozilla/go-cose#69
Correct README to reflect the defork from mozilla/go-cose#69yogeshbdeshpande merged 4 commits intomainfrom
Conversation
8c1dfc4 to
7d7f0fd
Compare
thomas-fossati
left a comment
There was a problem hiding this comment.
couple of grammar fixes
README.md
Outdated
| **Current Release**: [go-cose alpha 1][release-alpha-1] | ||
|
|
||
| The project was *initially* forked from the upstream [mozilla/go-cose][mozilla-go-cose] project, however it was agreed between the veraison and mozilla maintainers to retire mozilla/go-cose project and focus on [veraison/go-cose][veraison-go-cose] as the active project. | ||
| The project was *initially* forked from the upstream [mozilla-services/go-cose][mozilla-go-cose] project, however it was agreed between the Veraison and Mozilla maintainers to retire mozilla-services/go-cose project and focus on [veraison/go-cose][veraison-go-cose] as the active project. |
There was a problem hiding this comment.
I prefer using the active phrasing (i.e., as it was before) to the passive construction.
There was a problem hiding this comment.
Nothing wrong in introducing with a brief background of the initial state (earlier) and then thanking the original contributors (in present) and then moving on. In my view it flows well.
There was a problem hiding this comment.
what I am referring to is the change from the original:
the veraison and mozilla maintainers have agreed to [...]
to
it was agreed between the Veraison and Mozilla maintainers to [...]
(the former is better.)
There was a problem hiding this comment.
But that was correct when the action was in progress, like I have agreed to hand over XYZ to you.
But once the handover happened that should also go to past tense, like It was agreed between UK and EU to part ways back in 2019.
There was a problem hiding this comment.
But it's not that the agreement has vanished. Or so I hope :-)
There was a problem hiding this comment.
Hopefully this is viewed far better than brexit :)
|
@SteveLasker & @thomas-fossati Could you please approve the PR. Seems to be in pretty good shape. We need to make a release now once this is merged. |
|
Looks great. @thomas-fossati, if you could add the other LGTM, we could merge. |
|
Thanks @SteveLasker & @thomas-fossati for adding your comments and review. |
No description provided.