Skip to content

Fixes DCO signoff issue#548

Merged
Gregory-Pereira merged 1 commit intoinstructlab:release-1.0from
vishnoianil:dco-fix
Feb 4, 2025
Merged

Fixes DCO signoff issue#548
Gregory-Pereira merged 1 commit intoinstructlab:release-1.0from
vishnoianil:dco-fix

Conversation

@vishnoianil
Copy link
Member

Auth session keeps information from the user profile, so if the email address is not made visible on the profile or alias is set as an email address, UI was using that email address for signoff. In some cases, this was failing because that email address is not users primary email address.

For DCO to pass, user needs to signoff the commits with it's primary email address set in his account.

Auth session keeps information from the user profile,
so if the email address is not made visible on the profile
or alias is set as an email address, UI was using that email
address for signoff. In some cases, this was failing because
that email address is not users primary email address.

For DCO to pass, user needs to signoff the commits with it's
primary email address set in his account.

Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
@vishnoianil vishnoianil requested a review from nerdalert February 4, 2025 05:13
Copy link
Collaborator

@Gregory-Pereira Gregory-Pereira left a comment

Choose a reason for hiding this comment

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

/lgtm

@Gregory-Pereira Gregory-Pereira merged commit 23ae8b9 into instructlab:release-1.0 Feb 4, 2025
5 checks passed
@vishnoianil vishnoianil deleted the dco-fix branch February 4, 2025 05:21
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.

2 participants