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.

Sign up for GitHub

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

Jump to bottom

Rock CI: first pack the rock, then install the rest of the deps #264

Open

sed-i opened this issue Feb 11, 2025 · 0 comments

Contributor

sed-i commented Feb 11, 2025

Currently, if packing the rock fails, we discover it only after 5 minutes of installing dependencies.

observability/.github/workflows/rock-pull-request.yaml

Lines 51 to 57 in 3b9dbca

    
                 - name: Pack and test the rocks 
        
                   run: | 
        
                     for version in ${{ needs.changes.outputs.versions }}; do 
        
                       just pack "$version" 
        
                       just test "$version" 
        
                       just clean "$version" 
        
                     done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment