Skip to content

add a task for looking at an info object - #959

Merged
johrstrom merged 1 commit into
masterfrom
info-task
Aug 20, 2026
Merged

add a task for looking at an info object#959
johrstrom merged 1 commit into
masterfrom
info-task

Conversation

@johrstrom

Copy link
Copy Markdown
Contributor

Please review our Contributing Guide before submitting a pull request.

What does this PR do?

Adds a task to print OodCore::Info object in json format. While reviewing an upstream ondemand request I wondered about the format of certain fields, specifically time fields and how they were formatted. This task helps generate an object to inspect a given fields format.

Related issue

Does not close any issue.

Testing

  • Tests included
  • No tests needed — reason: it's a rake task

Checklist

  • Follows project code style and conventions
  • Documentation provided (if new feature, adapter or behavior change)
  • This is a large feature and was discussed in an issue first (if applicable)

Anything else?

Screenshots, context, or anything reviewers should pay close attention to.

@Bubballoo3 Bubballoo3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! Very useful too as it reminded me to include total memory in to_h as part of #958!

@johrstrom

Copy link
Copy Markdown
Contributor Author

it reminded me to include total memory in to_h as part of #958!

Should probably refactor that to instance_variables.to_h?

@johrstrom
johrstrom merged commit 9627dd2 into master Aug 20, 2026
4 checks passed
@johrstrom
johrstrom deleted the info-task branch August 20, 2026 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants