Skip to content

build/test: some fixups for the google import.#859

Merged
mattklein123 merged 3 commits into
envoyproxy:masterfrom
htuch:import-fixups
Apr 28, 2017
Merged

build/test: some fixups for the google import.#859
mattklein123 merged 3 commits into
envoyproxy:masterfrom
htuch:import-fixups

Conversation

@htuch
Copy link
Copy Markdown
Member

@htuch htuch commented Apr 28, 2017

  • Fix TestEnvironment::runfilesDirectory() to use TEST_RUNDIR, rather
    than the other way around.

  • Remove use of diff -Z in hotrestart_test, since it's not needed and
    isn't supported by all versions of diff.

  • Add license directives to all files.

* Fix TestEnvironment::runfilesDirectory() to use TEST_RUNDIR, rather
  than the other way around.

* Remove use of diff -Z in hotrestart_test, since it's not needed and
  isn't supported by all versions of diff.

* Add license directives to all files.
@mattklein123
Copy link
Copy Markdown
Member

Can we enforce "licenses(["notice"]) # Apache 2" in check/fix format? I fear this will end up getting dropped occasionally.

@mattklein123 mattklein123 merged commit ac817d5 into envoyproxy:master Apr 28, 2017
mathetake added a commit that referenced this pull request Mar 3, 2026
**Description**

Since the next EG v1.5 will be our target version that AIGW v0.3 will be
based on, there's no reason to keep it at the moment. This is mainly to
unblock the incoming feature patches that will rely on v1.5 feature,
such as #823.

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.

2 participants