Skip to content

Commit

Permalink
Fix DepsNotifyUpdatesAvailable logging event (#7262)
Browse files Browse the repository at this point in the history
  • Loading branch information
gshank authored and stu-k committed Apr 5, 2023
1 parent 6a4c3eb commit d5ebbdc
Show file tree
Hide file tree
Showing 4 changed files with 816 additions and 823 deletions.
7 changes: 1 addition & 6 deletions core/dbt/events/types.proto
Original file line number Diff line number Diff line change
Expand Up @@ -57,11 +57,6 @@ message ReferenceKeyMsg {
string identifier = 3;
}

// ListOfStrings
message ListOfStrings {
repeated string value = 1;
}

// GenericMessage, used for deserializing only
message GenericMessage {
EventInfo info = 1;
Expand Down Expand Up @@ -1301,7 +1296,7 @@ message DepsListSubdirectoryMsg {

// M019
message DepsNotifyUpdatesAvailable {
ListOfStrings packages = 1;
repeated string packages = 1;
}

message DepsNotifyUpdatesAvailableMsg {
Expand Down
2 changes: 1 addition & 1 deletion core/dbt/events/types.py
Original file line number Diff line number Diff line change
Expand Up @@ -1230,7 +1230,7 @@ def code(self):
return "M019"

def message(self) -> str:
return f"Updates available for packages: {self.packages.value} \
return f"Updates available for packages: {self.packages} \
\nUpdate your versions in packages.yml, then run dbt deps"


Expand Down
1,628 changes: 813 additions & 815 deletions core/dbt/events/types_pb2.py

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion tests/unit/test_events.py
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,7 @@ def test_event_codes(self):
types.DepsUpdateAvailable(version_latest=""),
types.DepsUpToDate(),
types.DepsListSubdirectory(subdirectory=""),
types.DepsNotifyUpdatesAvailable(packages=[]),
types.DepsNotifyUpdatesAvailable(packages=["my_pkg", "other_pkg"]),
types.RetryExternalCall(attempt=0, max=0),
types.RecordRetryException(exc=""),
types.RegistryIndexProgressGETRequest(url=""),
Expand Down

0 comments on commit d5ebbdc

Please sign in to comment.