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

Fixed Typo #11222

Merged
merged 2 commits into from
Jun 3, 2022
Merged

Fixed Typo #11222

merged 2 commits into from
Jun 3, 2022

Conversation

PrajwalBorkar
Copy link
Contributor

@PrajwalBorkar PrajwalBorkar commented May 30, 2022

What it does

Closes #11218

How to test

Confirm the description update is correct.

Review checklist

Reminder for reviewers

Signed-off-by: Prajwal Borkar [email protected]

@PrajwalBorkar PrajwalBorkar changed the title Fixed Typo Fixed Typo May 30, 2022
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution 👍

In order to accept your changes please be sure to sure to sign the eclipse contributor agreement (eca) with the same email as your authorship.

@PrajwalBorkar
Copy link
Contributor Author

Thank you for your contribution 👍

In order to accept your changes please be sure to sure to sign the eclipse contributor agreement (eca) with the same email as your authorship.

Should I create account in Eclipse as doc suggests or just signed off line in the commit msg will work?

@vince-fugnitto
Copy link
Member

Should I create account in Eclipse as doc suggests or just signed off line in the commit msg will work?

You'll need to follow the docs and create an eclipse account yes.

@PrajwalBorkar
Copy link
Contributor Author

Should I create account in Eclipse as doc suggests or just signed off line in the commit msg will work?

You'll need to follow the docs and create an eclipse account yes.

Done. But changes not reflected here.

@vince-fugnitto
Copy link
Member

@JonasHelming any objection to the authorship email and eca? In the past I don't think we allowed noreply emails:

From 6ab8a14d925a3a48ff07b0e1df94abc8a1f51e5b Mon Sep 17 00:00:00 2001
From: Prajwal Borkar <[email protected]>
Date: Mon, 30 May 2022 22:53:17 +0530
Subject: [PATCH] Update proxy-cli-contribution.ts

---
 packages/core/src/node/request/proxy-cli-contribution.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/core/src/node/request/proxy-cli-contribution.ts b/packages/core/src/node/request/proxy-cli-contribution.ts
index 4b034d4d1478..bde6a864437b 100644
--- a/packages/core/src/node/request/proxy-cli-contribution.ts
+++ b/packages/core/src/node/request/proxy-cli-contribution.ts
@@ -40,7 +40,7 @@ export class ProxyCliContribution implements CliContribution {
             type: 'string'
         });
         conf.option(StrictSSL, {
-            description: 'Detemines whether SSL is strictly set for outgoing requests.',
+            description: 'Determines whether SSL is strictly set for outgoing requests.',
             type: 'boolean'
         });
     }

@JonasHelming
Copy link
Contributor

@PrajwalBorkar Thank you for you first contribution! Please update your commit to origin from the same E-Mail you used for the Eclipse account. It should be your real mail address please. The background for this procedure is the IP check for this open source projects. To enable adopters to use Theia, we need to ensure that all commits to the project are complying with the ECA.

@vince-fugnitto
Copy link
Member

@PrajwalBorkar do you still plan on contributing the fix?

@PrajwalBorkar
Copy link
Contributor Author

PrajwalBorkar commented Jun 2, 2022

@PrajwalBorkar do you still plan on contributing the fix?

I already made the changes. Do I need to do anything now?

@vince-fugnitto
Copy link
Member

@PrajwalBorkar as #11222 (comment) mentioned it looks like you are using a noreply email which I don't believe we allow:

From: Prajwal Borkar <[email protected]>

it should be your real mail address please.

@PrajwalBorkar
Copy link
Contributor Author

PrajwalBorkar commented Jun 2, 2022

@PrajwalBorkar as #11222 (comment) mentioned it looks like you are using a noreply email which I don't believe we allow:

From: Prajwal Borkar <[email protected]>

it should be your real mail address please.

Now how to change that? Should I open new PR instead?

@msujew
Copy link
Member

msujew commented Jun 2, 2022

Now how to change that? Should I open new PR instead?

@PrajwalBorkar No need to create a new PR. You can amend your commit using another author, something like this:

git commit --amend --author="John Doe <[email protected]>"

You of course need to change the associated email for the ECA as well.

git commit --amend --author="Prajwal Borkar <[email protected]>"
@PrajwalBorkar
Copy link
Contributor Author

Now how to change that? Should I open new PR instead?

@PrajwalBorkar No need to create a new PR. You can amend your commit using another author, something like this:

git commit --amend --author="John Doe <[email protected]>"

You of course need to change the associated email for the ECA as well.

Done. Please review for any changes.

Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

The changes look good to me, as well as the authorship 👍
Thank you for your first contribution and patience! We look forward to any future contributions.

@vince-fugnitto vince-fugnitto merged commit bbac7e3 into eclipse-theia:master Jun 3, 2022
Copy link

@randydwhitephotography randydwhitephotography left a comment

Choose a reason for hiding this comment

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

merged

@vince-fugnitto vince-fugnitto added this to the 1.27.0 milestone Jun 30, 2022
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.

request: fix typo in description
5 participants