Skip to content
This repository has been archived by the owner on Mar 8, 2020. It is now read-only.

Commit

Permalink
Merge pull request #191 from juanjux/fix/remove_docgen
Browse files Browse the repository at this point in the history
Remove the (removed) docgen Makefile target
  • Loading branch information
juanjux authored Oct 5, 2017
2 parents 09a4fc2 + 45bb510 commit a130294
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 21 deletions.
13 changes: 2 additions & 11 deletions assets/build/bindata.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

11 changes: 1 addition & 10 deletions etc/build/make/rules.mk
Original file line number Diff line number Diff line change
Expand Up @@ -60,10 +60,6 @@ ALLOWED_IN_DOCKERFILE = \
BUILD_USER BUILD_UID BUILD_PATH \
DOCKER_IMAGE DOCKER_IMAGE_VERSIONED DOCKER_BUILD_NATIVE_IMAGE

# generated documentation
DOCGEN_FILES += \
ANNOTATION.md

# we export the variable to allow envsubst, substitute the vars in the
# Dockerfiles
export
Expand Down Expand Up @@ -96,7 +92,7 @@ test-driver-internal: build-native-internal
@cd driver; \
$(RUN_VERBOSE) $(GO_TEST) ./...

build: docgen | build-native build-driver $(DOCKER_IMAGE_VERSIONED)
build: | build-native build-driver $(DOCKER_IMAGE_VERSIONED)
build-native: $(BUILD_PATH) $(DOCKER_BUILD_NATIVE_IMAGE)
@$(RUN) $(BUILD_NATIVE_CMD) make build-native-internal

Expand Down Expand Up @@ -130,11 +126,6 @@ push: build
$(RUN) $(DOCKER_PUSH) $(call unescape_docker_tag,$(DOCKER_IMAGE):latest); \
fi;

docgen: $(DOCGEN_FILES)

ANNOTATION.md: driver/normalizer/annotation.go
@$(GO_RUN) driver/main.go docgen > $@

validate:
@$(RUN) $(bblfsh-sdk) update --dry-run

Expand Down

0 comments on commit a130294

Please sign in to comment.