Skip to content

Windows: Add note regarding the usage of ./ci/run_envoy_docker#15820

Merged
mattklein123 merged 2 commits intoenvoyproxy:mainfrom
davinci26:windowsBuildNote
Apr 8, 2021
Merged

Windows: Add note regarding the usage of ./ci/run_envoy_docker#15820
mattklein123 merged 2 commits intoenvoyproxy:mainfrom
davinci26:windowsBuildNote

Conversation

@davinci26
Copy link
Copy Markdown
Member

Signed-off-by: Sotiris Nanopoulos <sonanopo@,microsoft.com>

Windows Build: Add note regarding the usage of ./ci/run_envoy_docker.sh './ci/windows_ci_steps.sh'

…sh './ci/windows_ci_steps.sh'

Signed-off-by: davinci26 <sotirisnan@gmail.com>
Signed-off-by: davinci26 <sotirisnan@gmail.com>
@davinci26
Copy link
Copy Markdown
Member Author

@wrowe as discussed offline

cc @envoyproxy/windows-dev

@sunjayBhatia
Copy link
Copy Markdown
Member

Probably also worth eventually emulating the Linux helper script pattern or maybe even merging the two

@davinci26
Copy link
Copy Markdown
Member Author

We already have #15819

@wrowe wrowe self-assigned this Apr 2, 2021
Copy link
Copy Markdown
Contributor

@wrowe wrowe 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 creating an issue for follow-up.

@davinci26
Copy link
Copy Markdown
Member Author

@envoyproxy/senior-maintainers can someone merge this small doc change?

@mattklein123 mattklein123 merged commit cd9d19e into envoyproxy:main Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants