Skip to content

Commit

Permalink
schema dsl: make Property a case class... (#292)
Browse files Browse the repository at this point in the history
... so it can be used nicely in user feedback. Current example _before_
this PR:
```
[error] (schema / Compile / runMain) java.lang.AssertionError: proto ids must be unique across all schema elements, however we found the following duplicates in node properties:
[error] 251 -> flatgraph.schema.Property@52d99cc,flatgraph.schema.Property@a415a1d
[error] (schema / Compile / runMain) java.lang.AssertionError: proto ids must be unique across all schema elements, however we found the following duplicates in node properties:
[error] 251 -> flatgraph.schema.Property@3ab75944,flatgraph.schema.Property@34fe78a8
```
  • Loading branch information
mpollmeier authored Jan 14, 2025
1 parent b5ec183 commit 5ca8dba
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,7 @@ object EdgeType {
}
}

class Property[A](val name: String, val valueType: Property.ValueType[A], val comment: Option[String] = None, val schemaInfo: SchemaInfo)
case class Property[A](name: String, valueType: Property.ValueType[A], comment: Option[String] = None, schemaInfo: SchemaInfo)
extends HasClassName
with HasOptionalProtoId
with HasSchemaInfo {
Expand Down

0 comments on commit 5ca8dba

Please sign in to comment.