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

change git repo link for where examples are #35949

Merged
merged 2 commits into from
Sep 3, 2024
Merged

Conversation

DavidGoodwin
Copy link
Contributor

see also #32737 and #32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to the right github repo (examples)

Copy link

Hi @DavidGoodwin, welcome and thank you for your contribution.

We will try to review your Pull Request as quickly as possible.

In the meantime, please take a look at the contribution guidelines if you have not done so already.

🐱

Caused by: #35949 was opened by DavidGoodwin.

see: more, trace.

Copy link
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.

thanks for the follow up

one small change and this should be good to land, thanks

docs/root/start/sandboxes/setup.rst Outdated Show resolved Hide resolved
@phlax
Copy link
Member

phlax commented Sep 3, 2024

@DavidGoodwin could you fix DCO please

@DavidGoodwin

This comment was marked as resolved.

Signed-off-by: David Goodwin <[email protected]>
Copy link
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 @DavidGoodwin for the headsup and contribution

@phlax phlax enabled auto-merge (squash) September 3, 2024 11:51
@DavidGoodwin
Copy link
Contributor Author

@phlax thanks for your patience :)

@phlax phlax merged commit 69da813 into envoyproxy:main Sep 3, 2024
29 checks passed
phlax pushed a commit to phlax/envoy that referenced this pull request Sep 20, 2024
see also envoyproxy#32737 and
envoyproxy#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit to phlax/envoy that referenced this pull request Sep 20, 2024
see also envoyproxy#32737 and
envoyproxy#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit that referenced this pull request Sep 20, 2024
see also #32737 and
#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit to phlax/envoy that referenced this pull request Sep 20, 2024
see also envoyproxy#32737 and
envoyproxy#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit to phlax/envoy that referenced this pull request Sep 20, 2024
see also envoyproxy#32737 and
envoyproxy#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit to phlax/envoy that referenced this pull request Sep 20, 2024
see also envoyproxy#32737 and
envoyproxy#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit to phlax/envoy that referenced this pull request Sep 20, 2024
see also envoyproxy#32737 and
envoyproxy#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit to phlax/envoy that referenced this pull request Sep 20, 2024
see also envoyproxy#32737 and
envoyproxy#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit to phlax/envoy that referenced this pull request Sep 20, 2024
see also envoyproxy#32737 and
envoyproxy#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit to phlax/envoy that referenced this pull request Sep 20, 2024
see also envoyproxy#32737 and
envoyproxy#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit that referenced this pull request Sep 20, 2024
see also #32737 and
#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
phlax pushed a commit that referenced this pull request Sep 23, 2024
see also #32737 and
#32737 (comment),

Commit Message: update git repo url in docs to refer to the new examples
repo
Additional Description: might fix docs
Risk Level: none
Testing: none
Docs Changes: changes docs/root/start/sandboxes/setup.rst to refer to
the right github repo (examples)

---------

Signed-off-by: David Goodwin <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
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