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 2451-modules-compat.md fix typo #2

Closed
wants to merge 1 commit into from

Conversation

extemporalgenome
Copy link

No description provided.

Signed-off-by: Kevin Gillette <extemporalgenome@gmail.com>
cueckoo pushed a commit that referenced this pull request Jun 3, 2024
Spotted by extemporalgenome in #2 and #3 but because of Gerrit being
strict that the commit email addressed and "Signed-Of-By" are the same
it was quicker/simpler to make the changes again.

Signed-off-by: Paul Jolly <paul@myitcv.io>
Change-Id: I6d85fef64939c6f920c48c594a7ec844fe06b5c8
cueckoo pushed a commit that referenced this pull request Jun 3, 2024
Spotted by extemporalgenome in #2 and #3 but because of Gerrit being
strict that the commit email addressed and "Signed-Of-By" are the same
it was quicker/simpler to make the changes again.

Signed-off-by: Paul Jolly <paul@myitcv.io>
Change-Id: I6d85fef64939c6f920c48c594a7ec844fe06b5c8
Reviewed-on: https://review.gerrithub.io/c/cue-lang/proposal/+/1195638
Reviewed-by: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: CUEcueckoo <cueckoo@cuelang.org>
@myitcv
Copy link
Member

myitcv commented Jun 3, 2024

Thanks for taking the time to submit these PRs. It seems GerritHub is more strict that GitHub's DCO check; GerritHub complained about the difference between your commit author email address and the "Signed-Off-By". Rather than make you go through a dance of fixing that up, I took the liberty of incorporating both changes into https://review.gerrithub.io/c/cue-lang/proposal/+/1195638

@myitcv myitcv closed this Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants