Skip to content

rhdh customized prompt #567

Merged
tisnik merged 4 commits into
road-core:mainfrom
yangcao77:validator-prompt-rhdh
May 5, 2025
Merged

rhdh customized prompt #567
tisnik merged 4 commits into
road-core:mainfrom
yangcao77:validator-prompt-rhdh

Conversation

@yangcao77
Copy link
Copy Markdown
Contributor

Description

This PR creates rhdh customized prompt folder

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change

Related Tickets & Documents

https://issues.redhat.com/browse/RHDHPAI-747
https://issues.redhat.com/browse/RHDHPAI-748
https://issues.redhat.com/browse/RHDHPAI-749

Checklist before requesting a review

  • I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

Comment thread ols/customize/rhdh/keywords.py Outdated
Comment thread ols/customize/rhdh/keywords.py
Comment thread ols/customize/rhdh/prompts.py Outdated
Comment thread ols/customize/rhdh/keywords.py
Comment thread ols/customize/rhdh/keywords.py Outdated
yangcao77 added 2 commits May 5, 2025 13:03
Signed-off-by: Stephanie <yangcao@redhat.com>
Signed-off-by: Stephanie <yangcao@redhat.com>
@yangcao77 yangcao77 changed the title [WIP]rhdh customized prompt rhdh customized prompt May 5, 2025
@yangcao77
Copy link
Copy Markdown
Contributor Author

@tisnik PTAL on this PR, adding customized prompt folder for rhdh

Copy link
Copy Markdown
Collaborator

@tisnik tisnik left a comment

Choose a reason for hiding this comment

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

nice one!

@tisnik tisnik merged commit c102fe1 into road-core:main May 5, 2025
16 checks passed
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.

4 participants