Skip to content

ci: Remove unused submodules config#24211

Merged
phlax merged 1 commit intoenvoyproxy:mainfrom
phlax:ci-no-submodules
Nov 30, 2022
Merged

ci: Remove unused submodules config#24211
phlax merged 1 commit intoenvoyproxy:mainfrom
phlax:ci-no-submodules

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Nov 25, 2022

this also allows the the checkout action to be updated

Signed-off-by: Ryan Northey ryan@synca.io

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@phlax phlax changed the title ci: Remove unused submodules config [WIP] ci: Remove unused submodules config Nov 25, 2022
@phlax phlax marked this pull request as draft November 25, 2022 09:01
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Nov 25, 2022

/wait for #24212 to ensure changes are properly tested

@phlax phlax changed the title [WIP] ci: Remove unused submodules config ci: Remove unused submodules config Nov 28, 2022
@phlax phlax marked this pull request as ready for review November 28, 2022 14:43
@phlax phlax force-pushed the ci-no-submodules branch 2 times, most recently from 5d324c7 to 5e28082 Compare November 29, 2022 14:04
this also allows the the checkout action to be updated

Signed-off-by: Ryan Northey <ryan@synca.io>
Copy link
Copy Markdown
Contributor

@jpsim jpsim left a comment

Choose a reason for hiding this comment

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

Nice to remove this

@phlax phlax enabled auto-merge (squash) November 29, 2022 16:11
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Nov 30, 2022

/retest

@repokitteh-read-only
Copy link
Copy Markdown

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

🐱

Caused by: a #24211 (comment) was created by @phlax.

see: more, trace.

@phlax phlax merged commit 3f82c96 into envoyproxy:main Nov 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.

2 participants