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

Resolver 1.x #7747

Closed
Closed

Conversation

martinfrances107
Copy link
Contributor

This PR resolves a warning seen while running cargo update

warning: some crates are on edition 2021 which defaults to resolver = "2", but virtual workspaces default to resolver = "1"
note: to keep the current resolver, specify workspace.resolver = "1" in the workspace root's manifest
note: to use the edition 2021 resolver, specify workspace.resolver = "2" in the workspace root's manifest

@martinfrances107 martinfrances107 requested a review from a team as a code owner September 4, 2023 15:22
@amrbashir
Copy link
Member

The commit is still not signed 😅

@amrbashir amrbashir mentioned this pull request Sep 4, 2023
13 tasks
warning: some crates are on edition 2021 which defaults to `resolver = "2"`, but virtual workspaces default to `resolver = "1"`
note: to keep the current resolver, specify `workspace.resolver = "1"` in the workspace root's manifest
note: to use the edition 2021 resolver, specify `workspace.resolver = "2"` in the workspace root's manifest
@martinfrances107
Copy link
Contributor Author

martinfrances107 commented Sep 4, 2023

@amrbashir sorry to give you grief .. It seems I am all fingers and thumbs ... when contributing to a project new to me.

when I run

git log --show-signature

-- it now looks ok

commit 838de10 (HEAD -> resolver_1px, me/resolver_1px)
Good "git" signature for [email protected] with ED25519 key SHA256:29wbwo9YfOgSImgSxJ0uwO0JTFnz6ZaiF8YOC6wamhc
Author: martinfrances107 [email protected]
Date: Mon Sep 4 15:19:42 2023 +0100

chore: resolves a warning seen while running cargo update

warning: some crates are on edition 2021 which defaults to `resolver = "2"`, but virtual workspaces default to `resolver = "1"`
note: to keep the current resolver, specify `workspace.resolver = "1"` in the workspace root's manifest
note: to use the edition 2021 resolver, specify `workspace.resolver = "2"` in the workspace root's manifest

@amrbashir
Copy link
Member

The commit still shows as unverified
image
maybe you didn't add the signature in your account settings on github?

@martinfrances107
Copy link
Contributor Author

martinfrances107 commented Sep 5, 2023

That is a ssh key I have had posted on the site for more than 5 years

Looking to see what is different rather than just plain wrong...I am signing with SSH where as every other commit to the code base is signed with a GPG key

I see that GPG is better than ssh in that it can be revoked

So I am going to setup a GPG key one sec..

Generating a GPG signing key is more involved than generating an SSH key, but GPG has features that SSH does not. A GPG key can expire or be revoked when no longer used. GitHub shows commits that were signed with such a key as "Verified" unless the key was marked as compromised. SSH keys don't have this capability.

@FabianLars
Copy link
Member

I also use my ssh key on one of my machines for commit signing. One weird gatcha is that you have to add the key to github twice, as an authentication key and as a signing key. Being able to use it as a signing key is a relatively new feature so if you didn't update it for 5 years github probably doesn't recognize it as a signing key yet.

@martinfrances107
Copy link
Contributor Author

@FabianLars thank you very much for the advice ... in the end I created a new PR which I can see has the verified badge. ( #7753)

I dropped the SSH approach and followed the GPG key tutorials

@martinfrances107 martinfrances107 deleted the resolver_1px branch September 10, 2023 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 📬Proposal
Development

Successfully merging this pull request may close these issues.

3 participants