Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: added genApi step before build image #235

Merged
merged 1 commit into from
Oct 24, 2024
Merged

fix: added genApi step before build image #235

merged 1 commit into from
Oct 24, 2024

Conversation

Pugma
Copy link
Collaborator

@Pugma Pugma commented Oct 24, 2024

User description

確認お願いします〜

release.yaml と同じ方法に変更しただけなのでチェックは簡単だと思います


PR Type

enhancement


Description

  • 新しいジョブ genApi を追加し、API生成をDockerビルドから分離しました。
  • genApi ジョブでAPIを生成し、生成されたAPIをアーティファクトとしてアップロードするようにしました。
  • build-preview-image ジョブで生成されたAPIをダウンロードして使用するように変更しました。
  • build-preview-image ジョブが genApi ジョブに依存するように設定し、ビルドプロセスを改善しました。

Changes walkthrough 📝

Relevant files
Enhancement
preview.yaml
API生成ステップを追加し、ビルドプロセスを改善                                                                 

.github/workflows/preview.yaml

  • 新しいジョブ genApi を追加し、API生成をDockerビルドから分離
  • genApi ジョブでAPIを生成し、生成されたAPIをアーティファクトとしてアップロード
  • build-preview-image ジョブで生成されたAPIをダウンロードして使用
  • build-preview-image ジョブが genApi ジョブに依存するように設定
  • +24/-0   

    💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

    @Pugma Pugma requested a review from cp-20 October 24, 2024 04:04
    @Pugma Pugma enabled auto-merge October 24, 2024 04:04
    Copy link

    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

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

    Dependency Management
    genApi ジョブが build-preview-image ジョブに依存していますが、依存関係の設定が正しく行われているか確認が必要です。

    Copy link

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Score
    Enhancement
    Node.jsのセットアップアクションでnpmキャッシュのパスを明確に設定することを提案します。

    actions/setup-node@v4withセクションにcache:
    'npm'
    を指定していますが、キャッシュの利用方法を明確にするためにcache-dependency-pathを指定することをお勧めします。

    .github/workflows/preview.yaml [20-23]

     - uses: actions/setup-node@v4
       with:
         node-version-file: .node-version
         cache: npm
    +    cache-dependency-path: 'package-lock.json'
    Suggestion importance[1-10]: 7

    Why: The suggestion to specify cache-dependency-path enhances the caching mechanism by ensuring the cache is correctly utilized based on the package-lock.json. This can improve build efficiency and reliability, making it a valuable enhancement.

    7

    @Luke256
    Copy link

    Luke256 commented Oct 24, 2024

    問題ないように見えます。

    jobsbuild-preview-imagerelease.yamlでのimageと同じなら、typo等はないです

    @Pugma Pugma disabled auto-merge October 24, 2024 05:43
    Copy link
    Member

    @Takeno-hito Takeno-hito left a comment

    Choose a reason for hiding this comment

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

    :blob_thumbs_up:

    @Pugma Pugma merged commit 14216c6 into main Oct 24, 2024
    10 of 11 checks passed
    @Pugma Pugma deleted the fix/previewFlow branch October 24, 2024 05:50
    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