Skip to content

DEVELOPER.md: add link to vscode docs#16537

Merged
phlax merged 4 commits intoenvoyproxy:mainfrom
daixiang0:win
May 27, 2021
Merged

DEVELOPER.md: add link to vscode docs#16537
phlax merged 4 commits intoenvoyproxy:mainfrom
daixiang0:win

Conversation

@daixiang0
Copy link
Copy Markdown
Member

@daixiang0 daixiang0 commented May 18, 2021

Signed-off-by: Long Dai long0dai@foxmail.com

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md

Commit Message:

It is hard for new developers to find how to setup vscode env, so add it.

Additional Description:
Risk Level: Low
Testing:
Docs Changes: Yes
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

@phlax phlax self-assigned this May 18, 2021
@daixiang0
Copy link
Copy Markdown
Member Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #16537 (comment) was created by @daixiang0.

see: more, trace.

@daixiang0
Copy link
Copy Markdown
Member Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #16537 (comment) was created by @daixiang0.

see: more, trace.

daixiang0 added 3 commits May 20, 2021 09:29
Signed-off-by: Long Dai <long0dai@foxmail.com>
Signed-off-by: Long Dai <long0dai@foxmail.com>
Signed-off-by: Long Dai <long0dai@foxmail.com>
@daixiang0
Copy link
Copy Markdown
Member Author

@phlax could it be merged?

Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @daixiang0

Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

hmm - actually @daixiang0 im looking at the document overall and thinking that this should maybe be in the list below

Signed-off-by: Long Dai <long0dai@foxmail.com>
@daixiang0
Copy link
Copy Markdown
Member Author

I update it, please review again.

Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm - thanks for updating @daixiang0

@daixiang0 daixiang0 requested a review from phlax May 27, 2021 08:38
@phlax phlax merged commit 0beee9e into envoyproxy:main May 27, 2021
@daixiang0 daixiang0 deleted the win branch May 28, 2021 07:32
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 2021
Signed-off-by: Long Dai <long0dai@foxmail.com>
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