Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

print options #98

Merged
merged 1 commit into from
Mar 29, 2023
Merged

print options #98

merged 1 commit into from
Mar 29, 2023

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Mar 29, 2023

Summary by OpenAI

Release Notes:

  • New Feature: Adds a print() method to the Options class to print all options using core.info().
  • Bug fix: None.

"Printing options with ease, no more guessing what's in these. With core.info() we can see, all the options that used to be unseen."

@github-actions
Copy link
Contributor

🤖 OpenAI

Thank you for letting me participate in the code review. Based on my analysis, the "print options" pull request adds a print statement to output options and includes a new print() method to the Options class to print all options using core.info().

Here is a summary of the changes made in each file:

File Summary
src/main.ts Adds a print statement to output options.
src/options.ts Adds a print() method to the Options class to print all options using core.info().

Overall, the changes seem straightforward and do not introduce any major issues. However, it would be helpful to have more information about why these changes were made and how they fit into the larger context of the project.


Chat with 🤖 OpenAI Bot (@openai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @openai in a reply.

Files ignored due to filter (1)

Ignored files

  • dist/index.js

@harjotgill harjotgill merged commit c826d63 into main Mar 29, 2023
@harjotgill harjotgill deleted the print_options branch March 29, 2023 01:51
shoyuf pushed a commit to maybeLab/openai-mr-reviewer-gltlab that referenced this pull request Jun 25, 2023
<!-- This is an auto-generated comment: release notes by openai -->
### Summary by OpenAI

Release Notes:

- New Feature: Adds a `print()` method to the `Options` class to print
all options using `core.info()`.
- Bug fix: None.

> "Printing options with ease, no more guessing what's in these. With
`core.info()` we can see, all the options that used to be unseen."
<!-- end of auto-generated comment: release notes by openai -->
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant