Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix
IdentityType
classification bug. (#699)
* Fix `IdentityType` classification bug. Right now all typedefs are automatically categorized as `DerivedEnumerationType`; this is wrong because identity types could also be in a typedef. Fixed this bug and added tests. * Remove debugging statement * Add coverage for proto generation of typedef identityref * Run go generate ./... for CI check (#703) * Run go generate ./... for CI check * Regenerate exampleoc but don't depend on it for tests * Delete `StoreRawSchema` from `GeneratorConfig` since it's not being used (#705) * Delete `StoreRawSchema` from `GeneratorConfig` since it's not being used I don't see the value of why we need this flag: is it to save memory? For `exampleoc` this variable took 27MB, a negligible amount. * Reorganize Code Generation flags and Delete `ygen.GeneratorConfig` (#706) * ygen now only has `IROptions`. * `CodeGenerator` in gogen/protogen now carry IROptions and their language-specific post-IR generation options. * Code generation specific flags that are not used during IR generation are moved to `gogen` and `protogen`, duplicating if used by both (e.g. CallerName). * `SkipEnumDeduplication` is moved from ParseOptions to TransformationOptions.
- Loading branch information
Showing
30 changed files
with
1,084 additions
and
837 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,30 +1,20 @@ | ||
# ygot Makefile | ||
# | ||
# This makefile is used by Travis CI to run tests against the ygot library. | ||
# This makefile is used by GitHub Actions CI to run tests against the ygot library. | ||
# | ||
ROOT_DIR:=$(shell dirname $(realpath $(lastword $(MAKEFILE_LIST)))) | ||
|
||
test: | ||
go test ./... | ||
generate: | ||
cd ${ROOT_DIR}/demo/getting_started && SRCDIR=${ROOT_DIR} go generate | ||
cd ${ROOT_DIR}/proto/ywrapper && SRCDIR=${ROOT_DIR} go generate | ||
cd $(ROOT_DIR)/proto/yext && SRCDIR=${ROOT_DIR} go generate | ||
cd $(ROOT_DIR)/demo/uncompressed && SRCDIR=${ROOT_DIR} go generate | ||
cd $(ROOT_DIR)/demo/protobuf_getting_started && SRCDIR=${ROOT_DIR} ./update.sh | ||
cd $(ROOT_DIR)/integration_tests/uncompressed && SRCDIR=${ROOT_DIR} go generate | ||
cd $(ROOT_DIR)/integration_tests/annotations/apb && SRCDIR=${ROOT_DIR} go generate | ||
cd $(ROOT_DIR)/integration_tests/annotations/proto2apb && SRCDIR=${ROOT_DIR} go generate | ||
go generate ./demo/getting_started | ||
go generate ./proto/ywrapper | ||
go generate ./proto/yext | ||
go generate ./demo/uncompressed | ||
go generate ./demo/protobuf_getting_started | ||
go generate ./integration_tests/uncompressed | ||
go generate ./integration_tests/annotations/apb | ||
go generate ./integration_tests/annotations/proto2apb | ||
clean: | ||
rm -f ${ROOT_DIR}/demo/getting_started/pkg/ocdemo/oc.go | ||
rm -f ${ROOT_DIR}/demo/uncompressed/pkg/demo/uncompressed.go | ||
deps: | ||
go get -t -d ./ygot | ||
go get -t -d ./ygen | ||
go get -t -d ./generator | ||
go get -t -d ./proto_generator | ||
go get -t -d ./exampleoc | ||
go get -t -d ./ytypes | ||
go get -t -d ./demo/gnmi_telemetry | ||
rm -f demo/getting_started/pkg/ocdemo/oc.go | ||
rm -f demo/uncompressed/pkg/demo/uncompressed.go | ||
install: deps generate | ||
all: clean deps generate test |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.