Skip to content
New issue

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

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

Already on GitHub? # to your account

Add more information to crawler output #49

Merged
merged 1 commit into from
Aug 19, 2024
Merged

Add more information to crawler output #49

merged 1 commit into from
Aug 19, 2024

Conversation

davidallendj
Copy link
Collaborator

This PR includes more usable information to be parsed by SMD to build component and component endpoints.

@davidallendj davidallendj added the enhancement New feature or request label Aug 15, 2024
@davidallendj davidallendj self-assigned this Aug 15, 2024
Name: rf_computersystem.Name,
Manufacturer: rf_computersystem.Manufacturer,
SystemType: string(rf_computersystem.SystemType),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason you are calling string() here? I notice the other right-hand values above and below don't have itand they are of type string.

Copy link
Collaborator Author

@davidallendj davidallendj Aug 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure, since I didn't add that myself. I think it was added automatically by gofmt. I'll take a closer look at it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After looking at the gofish code, I think it's to cast from SystemType to a string, since SystemType is the actual type of rf_computersystem.SystemType.

@davidallendj
Copy link
Collaborator Author

Probably worth mentioning that with the creation of central schemas, I'm going to repurpose this PR to import and use them rather than the structs here in magellan.

Copy link
Member

@alexlovelltroy alexlovelltroy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alexlovelltroy alexlovelltroy merged commit a33fd44 into main Aug 19, 2024
@alexlovelltroy
Copy link
Member

I merged this one. I think a new PR is in order for the schema changes

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants