Skip to content

Commit

Permalink
Mark 'verify-all' as deprecated, but leave it to make pass existing PRs
Browse files Browse the repository at this point in the history
  • Loading branch information
jprzychodzen committed Sep 30, 2020
1 parent a0288cb commit 056bb5c
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -14,16 +14,19 @@
# See the License for the specific language governing permissions and
# limitations under the License.

# TODO(jprzychodzen): Remove after all PRs with 'pull-perf-tests-verify-all' are closed
# Deprecated, use verify-all-dev for local development
.PHONY: verify-all
verify-all: test verify-boilerplate verify-flags
verify-all:
echo "Deprecated, use 'verify-all-dev' for local development"

# verify-lint-dev is convenience target for developers to run all checks
# at once locally
.PHONY: verify-all-dev
verify-all-dev: verify-all verify-dashboard verify-lint
verify-all-dev: test verify-all-python verify-dashboard verify-lint

.PHONY: verify-all-python
verify-all: verify-boilerplate verify-flags
verify-all-python: verify-boilerplate verify-flags

# TODO(oxddr): go-build.sh doesn't work at the moment decide whether we need this at all
# .PHONY: build
Expand Down

0 comments on commit 056bb5c

Please sign in to comment.