Skip to content

Comments

feat: improve the prompt based on last evaluation run#25

Merged
billybonks merged 1 commit intomasterfrom
feat/eval-framework
Mar 20, 2025
Merged

feat: improve the prompt based on last evaluation run#25
billybonks merged 1 commit intomasterfrom
feat/eval-framework

Conversation

@billybonks
Copy link
Contributor

basically did some work with evaluator and updated the prompt accordingly

Dropped chore as the term is too overloaded and is not part of the electron or anglular specs

Copy link

@managerbot-app managerbot-app bot left a comment

Choose a reason for hiding this comment

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

👍🏿 👎🏿 review comments so that we know what works and what does not work


Please follow these instructions to generate the commit message:

if such and such changes its docs b ut if commit also has such then its a feat

Choose a reason for hiding this comment

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

This line contains confusing/unclear text: 'if such and such changes its docs b ut if commit also has such then its a feat'. Consider clarifying or rewording this guidance for better clarity.

- fix: A bug fix these changes ARE NOT in the docs directory
- docs: Documentation only changes for example changes that happen in docs directory
- style: Changes that do not affect the code execution (e.g., white-space, formatting, missing semi-colons, etc.) usually done by linters
- refactor:

Choose a reason for hiding this comment

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

The refactor description is incomplete or poorly formatted. The sentence structure seems off with 'just how the code is written executed'.

- build: Changes that affect the build system or external dependencies
- ci:
- Changes to the CI configuration files and scripts
- these changes should not change the way that the code is built otherwise its build

Choose a reason for hiding this comment

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

The 'ci' description contains a grammatical error: 'its build' should be 'it's build'.

@billybonks billybonks merged commit 9287b06 into master Mar 20, 2025
2 checks passed
@billybonks billybonks deleted the feat/eval-framework branch March 20, 2025 06:29
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.

1 participant