Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix format for id property in graph output json #13964

Merged
merged 2 commits into from
May 25, 2023
Merged

Fix format for id property in graph output json #13964

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
444a7a3
convert to str
czoido May 25, 2023
1b07dd0
franchus review
czoido May 25, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion conans/client/graph/graph.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ def __repr__(self):
def serialize(self):
result = OrderedDict()
result["ref"] = self.ref.repr_notime() if self.ref is not None else "conanfile"
result["id"] = getattr(self, "id") # Must be assigned by graph.serialize()
result["id"] = str(getattr(self, "id")) # Must be assigned by graph.serialize()
czoido marked this conversation as resolved.
Show resolved Hide resolved
result["recipe"] = self.recipe
result["package_id"] = self.package_id
result["prev"] = self.prev
Expand Down