Skip to content

Make codeflash github actions cooler (i hope) #66

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

Merged
merged 1 commit into from
Mar 25, 2025

Conversation

misrasaurabh1
Copy link
Contributor

@misrasaurabh1 misrasaurabh1 commented Mar 24, 2025

PR Type

  • Enhancement

Description

  • Added emoji symbols in GitHub Actions job names.

  • Updated CLI command strings with emojis.

  • Standardized workflow step naming across files.


Changes walkthrough 📝

Relevant files
Enhancement
cmd_init.py
Add emoji to dependency manager setup messages                     

codeflash/cli_cmds/cmd_init.py

  • Inserted snake emojis in dependency manager strings.
  • Updated returned setup names with decorative emojis.
  • +2/-2     
    codeflash-optimize.yaml
    Update workflow steps with descriptive emoji names             

    .github/workflows/codeflash-optimize.yaml

  • Renamed optimization job to mention Python.
  • Added emojis to checkout, setup, install, and optimization steps.
  • +6/-5     
    codeflash-optimize.yaml
    Standardize workflow naming with emojis                                   

    codeflash/cli_cmds/workflows/codeflash-optimize.yaml

  • Updated job name for optimization without bot exclusion.
  • Added emoji-decorated names for checkout, dependency installation, and
    code optimization.
  • +5/-4     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • 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

    Template Placeholder

    Verify that the template placeholders (e.g., {{ working_directory }} and {{ setup_python_dependency_manager }}) are correctly resolved or substituted before the workflow runs.

    {{ working_directory }}
    steps:
      - name: 🛎️ Checkout
        uses: actions/checkout@v4
        with:
          fetch-depth: 0
      - {{ setup_python_dependency_manager }}
      - name: 📦 Install Dependencies
        run: {{ install_dependencies_command }}
      - name: ⚡️Codeflash Optimization
        run: {{ codeflash_command }}

    Copy link

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Copy link
    Contributor

    @alvin-r alvin-r left a comment

    Choose a reason for hiding this comment

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

    Very Cool, Saurabh! 😎😎

    @misrasaurabh1 misrasaurabh1 merged commit 8ef811e into main Mar 25, 2025
    16 checks passed
    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.

    2 participants