Skip to content

feat: combine slash command jobs into single job steps#266

Merged
Aaron ("AJ") Steers (aaronsteers) merged 3 commits into
mainfrom
devin/1737695851-use-forked-dispatched-action
Jan 24, 2025
Merged

feat: combine slash command jobs into single job steps#266
Aaron ("AJ") Steers (aaronsteers) merged 3 commits into
mainfrom
devin/1737695851-use-forked-dispatched-action

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot commented Jan 24, 2025

Use forked slash-command-dispatch with 'dispatched' boolean output

Changes

  1. Updated workflow to use Aaron's forked action (aaronsteers/slash-command-dispatch@aj/fix/add-dispatched-bool-output)
  2. Combined slash command jobs into steps within a single job for better efficiency
  3. Simplified unrecognized command handling using the new 'dispatched' output

Implementation Details

  • Using forked action from Aaron's branch that adds the new 'dispatched' boolean output
  • The 'dispatched' output indicates whether a slash command was successfully dispatched
  • Moved unrecognized command handling into the main job as additional steps
  • Simplified conditional logic to use steps.dispatch.outputs.dispatched directly
  • Removed unnecessary job outputs since we're using step outputs directly

Testing

This change requires merging to main before it can be tested, as it modifies GitHub Actions workflow configuration.

Link to Devin run: https://app.devin.ai/sessions/19bfcd3c77084c55b445d7ba46e5f65c

Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

Original prompt:

@Devin - In the CDK repo, can you modify our slash command dispatch workflow as follows: If someone enters a one-line comment that starts with `/`, let's assume they are trying to enter a slash command. If it doesn't match a supported command (including if they literally type `/help` - then post back a comment to them telling what slash commands are available.

@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions github-actions Bot added the enhancement New feature or request label Jan 24, 2025
Comment thread .github/workflows/slash_command_dispatch.yml Outdated
Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
@devin-ai-integration devin-ai-integration Bot changed the title feat: use forked slash-command-dispatch with 'dispatched' output refactor: combine slash command jobs into single job steps Jan 24, 2025
@devin-ai-integration devin-ai-integration Bot changed the title refactor: combine slash command jobs into single job steps feat: combine slash command jobs into single job steps Jan 24, 2025
Comment thread .github/workflows/slash_command_dispatch.yml
@aaronsteers Aaron ("AJ") Steers (aaronsteers) deleted the devin/1737695851-use-forked-dispatched-action branch January 24, 2025 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant