Skip to content

build: circle-ci os x config#1507

Merged
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
turbinelabs:circle-ci-integration
Aug 22, 2017
Merged

build: circle-ci os x config#1507
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
turbinelabs:circle-ci-integration

Conversation

@zuercher
Copy link
Copy Markdown
Member

Circle CI configuration for a Mac OS X CI environment. Once this is on master, we can configure a webhook to trigger this build in my employer's Circle CI account and through the magic of some glue code should be a status check for Mac OS build/test. GitHub should automatically default to not blocking merges based on this check, which is the desired configuration.

The shell code in the config that deals with PULL_REQUEST et al configures a remote in the build environment's git repository in order to fetch the git SHA referenced by the pull request and merge it into our fork's master. Our fork's master is kept in sync with lyft/envoy:master.

@ryancox
Copy link
Copy Markdown
Contributor

ryancox commented Aug 22, 2017

+1

@mattklein123 mattklein123 merged commit 171aec9 into envoyproxy:master Aug 22, 2017
@zuercher zuercher deleted the circle-ci-integration branch September 13, 2017 16:34
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Signed-off-by: Mike Schore <mike.schore@gmail.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Signed-off-by: Mike Schore <mike.schore@gmail.com>
Signed-off-by: JP Simard <jp@jpsim.com>
mathetake added a commit that referenced this pull request Mar 3, 2026
**Description**
The translators shouldn't have to be tied with ext_proc APIs since it's
none of their concerns. This commit decouples the translator package
from them so that we can easily use them in dynamic modules too.

**Related Issues/PRs (if applicable)**

Preparation for #90

---------

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.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.

3 participants