Merge pull request #164 from ChrisCummins/release-v0.2.0

Release v0.2.0

.bazelrc

@@ -1,29 +1,21 @@
 # See: https://bazel.build/versions/master/docs/bazel-user-manual.html#flag--compilation_mode
 
-# This flag is set by the bazel IntelliJ plugin. Set it here so that alternating
-# between command line and IntelliJ invocations does not reset the analysis cache.
-build --runs_per_test=1
-test --runs_per_test=1
-
-# This project uses C++14.
+# This project requires c++14.
+#
+# NOTE(cummins): This flag has to be pushed out to the copts argument of a
+# handful of cxx targets, grep for them when changing the standard.
+#
+# NOTE(cummins): LLVM 3.8 is not compatible with C++17 or later.
 build --cxxopt='-std=c++14'
 test --cxxopt='-std=c++14'
 
-# Promote unused variable warnings to errors.
-build --cxxopt='-Werror=unused-variable'
-test --cxxopt='-Werror=unused-variable'
-
-# Promote unused result warnings to errors.
-build --cxxopt='-Werror=unused-result'
-test --cxxopt='-Werror=unused-result'
-
-# Promote warnings about missing braces on if/else blocks to errors.
-build --cxxopt='-Werror=dangling-else'
-test --cxxopt='-Werror=dangling-else'
+# Workaround for broken grpc build on macOS.
+# See: https://github.com/bazelbuild/bazel/issues/4341
+build --copt -DGRPC_BAZEL_BUILD
+test --copt -DGRPC_BAZEL_BUILD
 
-# Default to continuing on build error. Override using --keep_going=false.
-build --keep_going
-test --keep_going
+# Show test error output. Override using --test_output={summary,full}.
+test --test_output=errors
 
 # Show test error output. Override using --test_output={summary,full}.
 test --test_output=errors

.bazelversion

@@ -1 +1 @@
-3.4.1
+4.0.0

.bettercodehub.yml

@@ -2,7 +2,10 @@
 # Configuration for Better Code analysis.
 # See https://bettercodehub.com/docs/configuration-manual
 exclude:
+    - /benchmarks/.*
     - /bin/.*
+    - /models/.*
+    - /programl/third_party/.*
     - /tasks/.*
     - /tests/.*
     - /third_party/.*

.clang-format

@@ -1,13 +1,13 @@
 ---
-Language:        Cpp
+Language: Cpp
 # BasedOnStyle:  Google
 AccessModifierOffset: -1
 AlignAfterOpenBracket: Align
 AlignConsecutiveMacros: false
 AlignConsecutiveAssignments: false
 AlignConsecutiveDeclarations: false
 AlignEscapedNewlines: Left
-AlignOperands:   true
+AlignOperands: true
 AlignTrailingComments: true
 AllowAllArgumentsOnNextLine: true
 AllowAllConstructorInitializersOnNextLine: true
@@ -25,22 +25,22 @@ AlwaysBreakTemplateDeclarations: Yes
 BinPackArguments: true
 BinPackParameters: true
 BraceWrapping:
-  AfterCaseLabel:  false
-  AfterClass:      false
-  AfterControlStatement: false
-  AfterEnum:       false
-  AfterFunction:   false
-  AfterNamespace:  false
-  AfterObjCDeclaration: false
-  AfterStruct:     false
-  AfterUnion:      false
-  AfterExternBlock: false
-  BeforeCatch:     false
-  BeforeElse:      false
-  IndentBraces:    false
-  SplitEmptyFunction: true
-  SplitEmptyRecord: true
-  SplitEmptyNamespace: true
+    AfterCaseLabel: false
+    AfterClass: false
+    AfterControlStatement: false
+    AfterEnum: false
+    AfterFunction: false
+    AfterNamespace: false
+    AfterObjCDeclaration: false
+    AfterStruct: false
+    AfterUnion: false
+    AfterExternBlock: false
+    BeforeCatch: false
+    BeforeElse: false
+    IndentBraces: false
+    SplitEmptyFunction: true
+    SplitEmptyRecord: true
+    SplitEmptyNamespace: true
 BreakBeforeBinaryOperators: None
 BreakBeforeBraces: Attach
 BreakBeforeInheritanceComma: false
@@ -50,48 +50,48 @@ BreakConstructorInitializersBeforeComma: false
 BreakConstructorInitializers: BeforeColon
 BreakAfterJavaFieldAnnotations: false
 BreakStringLiterals: true
-ColumnLimit:     100
-CommentPragmas:  '^ IWYU pragma:'
+ColumnLimit: 100
+CommentPragmas: '^ IWYU pragma:'
 CompactNamespaces: false
 ConstructorInitializerAllOnOneLineOrOnePerLine: true
 ConstructorInitializerIndentWidth: 4
 ContinuationIndentWidth: 4
 Cpp11BracedListStyle: true
 DeriveLineEnding: true
 DerivePointerAlignment: false
-DisableFormat:   false
+DisableFormat: false
 ExperimentalAutoDetectBinPacking: false
 FixNamespaceComments: true
 ForEachMacros:
-  - foreach
-  - Q_FOREACH
-  - BOOST_FOREACH
-IncludeBlocks:   Regroup
+    - foreach
+    - Q_FOREACH
+    - BOOST_FOREACH
+IncludeBlocks: Regroup
 IncludeCategories:
-  - Regex:           '^<ext/.*\.h>'
-    Priority:        2
-    SortPriority:    0
-  - Regex:           '^<.*\.h>'
-    Priority:        1
-    SortPriority:    0
-  - Regex:           '^<.*'
-    Priority:        2
-    SortPriority:    0
-  - Regex:           '.*'
-    Priority:        3
-    SortPriority:    0
-IncludeIsMainRegex: '([-_](test|unittest))?$'
+    - Regex: ^<ext/.*\.h>
+      Priority: 2
+      SortPriority: 0
+    - Regex: ^<.*\.h>
+      Priority: 1
+      SortPriority: 0
+    - Regex: ^<.*
+      Priority: 2
+      SortPriority: 0
+    - Regex: .*
+      Priority: 3
+      SortPriority: 0
+IncludeIsMainRegex: ([-_](test|unittest))?$
 IncludeIsMainSourceRegex: ''
 IndentCaseLabels: true
 IndentGotoLabels: true
 IndentPPDirectives: None
-IndentWidth:     2
+IndentWidth: 2
 IndentWrappedFunctionNames: false
 JavaScriptQuotes: Leave
 JavaScriptWrapImports: true
 KeepEmptyLinesAtTheStartOfBlocks: false
 MacroBlockBegin: ''
-MacroBlockEnd:   ''
+MacroBlockEnd: ''
 MaxEmptyLinesToKeep: 1
 NamespaceIndentation: None
 ObjCBinPackProtocolList: Never
@@ -108,35 +108,35 @@ PenaltyExcessCharacter: 1000000
 PenaltyReturnTypeOnItsOwnLine: 200
 PointerAlignment: Left
 RawStringFormats:
-  - Language:        Cpp
-    Delimiters:
-      - cc
-      - CC
-      - cpp
-      - Cpp
-      - CPP
-      - 'c++'
-      - 'C++'
-    CanonicalDelimiter: ''
-    BasedOnStyle:    google
-  - Language:        TextProto
-    Delimiters:
-      - pb
-      - PB
-      - proto
-      - PROTO
-    EnclosingFunctions:
-      - EqualsProto
-      - EquivToProto
-      - PARSE_PARTIAL_TEXT_PROTO
-      - PARSE_TEST_PROTO
-      - PARSE_TEXT_PROTO
-      - ParseTextOrDie
-      - ParseTextProtoOrDie
-    CanonicalDelimiter: ''
-    BasedOnStyle:    google
-ReflowComments:  true
-SortIncludes:    true
+    - Language: Cpp
+      Delimiters:
+          - cc
+          - CC
+          - cpp
+          - Cpp
+          - CPP
+          - c++
+          - C++
+      CanonicalDelimiter: ''
+      BasedOnStyle: google
+    - Language: TextProto
+      Delimiters:
+          - pb
+          - PB
+          - proto
+          - PROTO
+      EnclosingFunctions:
+          - EqualsProto
+          - EquivToProto
+          - PARSE_PARTIAL_TEXT_PROTO
+          - PARSE_TEST_PROTO
+          - PARSE_TEXT_PROTO
+          - ParseTextOrDie
+          - ParseTextProtoOrDie
+      CanonicalDelimiter: ''
+      BasedOnStyle: google
+ReflowComments: true
+SortIncludes: true
 SortUsingDeclarations: true
 SpaceAfterCStyleCast: false
 SpaceAfterLogicalNot: false
@@ -150,18 +150,17 @@ SpaceBeforeRangeBasedForLoopColon: true
 SpaceInEmptyBlock: false
 SpaceInEmptyParentheses: false
 SpacesBeforeTrailingComments: 2
-SpacesInAngles:  false
+SpacesInAngles: false
 SpacesInConditionalStatement: false
 SpacesInContainerLiterals: true
 SpacesInCStyleCastParentheses: false
 SpacesInParentheses: false
 SpacesInSquareBrackets: false
 SpaceBeforeSquareBrackets: false
-Standard:        Auto
+Standard: Auto
 StatementMacros:
-  - Q_UNUSED
-  - QT_REQUIRE_VERSION
-TabWidth:        8
-UseCRLF:         false
-UseTab:          Never
-...
+    - Q_UNUSED
+    - QT_REQUIRE_VERSION
+TabWidth: 8
+UseCRLF: false
+UseTab: Never

.github/ISSUE_TEMPLATE/--bug-report.md

@@ -0,0 +1,44 @@
+---
+name: "\U0001F41B Bug Report"
+about: Create a report to help us improve ProGraML
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+## 🐛 Bug
+
+<!-- A clear and concise description of what the bug is. -->
+
+## To Reproduce
+
+Steps to reproduce the behavior:
+
+1.
+1.
+1.
+
+<!-- If you have a code sample, error messages, stack traces, please provide it here as well -->
+
+## Expected behavior
+
+<!-- A clear and concise description of what you expected to happen. -->
+
+## Environment
+
+<!-- Please fill in this checklist: -->
+
+- ProGraML version (if building from source, use `git rev-parse HEAD`):
+- How you installed ProGraML (source, pip):
+- OS:
+- Python version:
+- Build command you used (if building from source):
+- GCC/clang version (if building from source):
+- Bazel version (if building from source):
+- Versions of any other relevant libraries:
+
+
+## Additional context
+
+<!-- Add any other context about the problem here. -->

.github/ISSUE_TEMPLATE/--documentation.md

@@ -0,0 +1,12 @@
+---
+name: "\U0001F4DA Documentation"
+about: Report a documentation issue
+title: ''
+labels: 'Documentation'
+assignees: ''
+
+---
+
+## 📚 Documentation
+
+<!-- A clear and concise description of the issue. -->

.github/ISSUE_TEMPLATE/--feature-request.md

@@ -0,0 +1,28 @@
+---
+name: "\U0001F680Feature Request"
+about: Submit a proposal/request for a new ProGraML feature
+title: ''
+labels: 'Enhancement'
+assignees: ''
+
+---
+
+## 🚀 Feature
+
+<!-- A clear and concise description of the feature proposal. -->
+
+## Motivation
+
+<!-- Please outline the motivation for the proposal. Is your feature request related to a problem? e.g., I'm always frustrated when [...]. If this is related to another GitHub issue, please link here too. -->
+
+## Pitch
+
+<!-- A clear and concise description of what you want to happen. -->
+
+## Alternatives
+
+<!-- A clear and concise description of any alternative solutions or features you've considered, if any. -->
+
+## Additional context
+
+<!-- Add any other context or screenshots about the feature request here. -->

.github/ISSUE_TEMPLATE/--questions-help-support.md

@@ -0,0 +1,16 @@
+---
+name: "❓Questions/Help/Support"
+about: Do you need support? We have resources.
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+## ❓ Questions and Help
+
+<!-- A clear and concise description of the question. -->
+
+## Additional Context
+
+<!-- Add any other context about the problem here. -->

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1 @@
+Fixes #{issue number}