Skip to content

Conversation

@Futadaruma
Copy link
Collaborator

@Futadaruma Futadaruma commented Apr 8, 2025

User description

以下のPRと同様の変更です
traPtitech/BOT_GPT#33


PR Type

enhancement, configuration changes


Description

  • プロキシを使用するための設定を追加

  • OPENAI_KEYOPENAI_PROXY_KEYに変更

  • OPENAI.API_BASEの設定を追加

  • モデル設定をgpt-4ogpt-4o-miniに変更


Changes walkthrough 📝

Relevant files
Configuration changes
PR-Agent.yaml
プロキシ使用のための設定とモデル変更                                                                             

.github/workflows/PR-Agent.yaml

  • プロキシ使用のための設定を追加
  • APIキーとモデル設定を変更
+5/-2     

Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @github-actions
    Copy link
    Contributor

    github-actions bot commented Apr 8, 2025

    PR Reviewer Guide 🔍

    (Review updated until commit a72daa8)

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    設定確認

    新しく追加された環境変数設定(OPENAI_PROXY_KEY, OPENAI_API_BASE, CONFIG_MODEL, CONFIG_MODEL_TURBO)が正しく反映されること、及び既存のシステムとの整合性が保たれているかを確認してください。

    uses: qodo-ai/[email protected]
    env:
      OPENAI_KEY: ${{ secrets.OPENAI_PROXY_KEY }}
      OPENAI_API_BASE: 'https://llm-proxy.trap.jp'
      CONFIG_MODEL: 'gpt-4o'
      CONFIG_MODEL_TURBO: 'gpt-4o-mini'

    @github-actions
    Copy link
    Contributor

    github-actions bot commented Apr 8, 2025

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Impact
    General
    環境変数名のドットを修正

    環境変数名にドットを使用すると、YAMLのパース時に問題が発生する可能性があります。代わりにアンダースコアを使用することを検討してください。

    .github/workflows/PR-Agent.yaml [30-31]

    -CONFIG.MODEL: 'gpt-4o'
    -CONFIG.MODEL_TURBO: 'gpt-4o-mini'
    +CONFIG_MODEL: 'gpt-4o'
    +CONFIG_MODEL_TURBO: 'gpt-4o-mini'
    Suggestion importance[1-10]: 8

    __

    Why: The suggestion addresses a potential issue with YAML parsing by recommending the replacement of dots with underscores in environment variable names. This change can prevent parsing errors and improve compatibility, making it a valuable improvement to the PR.

    Medium

    @Futadaruma Futadaruma removed request for Pugma and sh0go07 April 8, 2025 15:59
    @Futadaruma
    Copy link
    Collaborator Author

    yamlファイルの環境変数名にドットがあるとよくないのでアンダースコアに変更します
    Qodoのドキュメントも環境変数名にアンダースコアを使っていることを確認しました

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    Category **Suggestion                                                                                                                                    ** Impact
    General
    環境変数名のドットを修正
    Medium

    @Futadaruma
    Copy link
    Collaborator Author

    /review

    @github-actions
    Copy link
    Contributor

    Persistent review updated to latest commit a72daa8

    @Futadaruma Futadaruma requested review from Pugma and sh0go07 April 12, 2025 12:27
    Copy link
    Collaborator

    @Pugma Pugma left a comment

    Choose a reason for hiding this comment

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

    よさそうです!
    対応ありがとうございます

    不要になった secret は削除しておいてください

    @Futadaruma Futadaruma merged commit 893ba0e into main Apr 15, 2025
    10 checks passed
    @Futadaruma Futadaruma deleted the feat/use_proxy branch April 15, 2025 03:22
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    3 participants