Skip to content

Commit

Permalink
fix: not serialize generc
Browse files Browse the repository at this point in the history
  • Loading branch information
doctrino committed Nov 6, 2024
1 parent d4c92cf commit c732ce0
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
6 changes: 5 additions & 1 deletion cognite/neat/_issues/_base.py
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,11 @@ def _get_variables(self) -> tuple[dict[str, str], bool]:
def dump(self) -> dict[str, Any]:
"""Return a dictionary representation of the issue."""
variables = vars(self)
output = {to_camel(key): self._dump_value(value) for key, value in variables.items() if value is not None}
output = {
to_camel(key): self._dump_value(value)
for key, value in variables.items()
if not (value is None or key.startswith("_"))
}
output["NeatIssue"] = type(self).__name__
return output

Expand Down
5 changes: 5 additions & 0 deletions docs/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,11 @@ Changes are grouped as follows:
- `Fixed` for any bug fixes.
- `Security` in case of vulnerabilities.

## TBD
### Fixed
- Serializing `ResourceNotDefinedError` class no longer raises a `ValueError`. This happens when a `ResourceNotDefinedError`
is found, for example, when calling `neat.verify()`.

## [0.96.4] - 05-11-**2024**
### Fixed
- `neat.to.excel` or `neat.to.yaml` now correctly writes `ViewTypes` and `Edge` that do not have the default
Expand Down

0 comments on commit c732ce0

Please sign in to comment.