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

feat(feature dev): Add setting to allow Q to run code and test commands #5853

Conversation

willyyhuang
Copy link
Contributor

@willyyhuang willyyhuang commented Oct 24, 2024

Problem

Q /dev team is launching a new feature: Allow Q to execute build/test based on customer's configuration 🎉🎉

Solution

We have made a couple tweaks to incorporate the user experience

  • Added a new setting to allow Q to run code and test commands per project

image

  • Added a pre-filled prompt option for customer to generate a configuration for this feature
  • Added a couple follow up buttons throughout the /dev flow and added logic around them

image

  • Updated copies
  • Updated codefileExtensions set

License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

Copy link

This pull request modifies code in src/ but no tests were added/updated. Confirm whether tests should be added or ensure the PR description explains why tests are not required.

@willyyhuang willyyhuang changed the title feature(feature dev): Add setting to opt in/out of QSDA auto build feature feat(feature dev): Add setting to opt in/out of QSDA auto build feature Oct 24, 2024
@willyyhuang willyyhuang force-pushed the wlsun/add-auto-build-feature-opt-in-out branch 2 times, most recently from 13eb178 to 39f4bc4 Compare October 25, 2024 15:47
@willyyhuang willyyhuang force-pushed the wlsun/add-auto-build-feature-opt-in-out branch from afcfa35 to d987ed4 Compare November 5, 2024 18:24
Copy link

github-actions bot commented Nov 5, 2024

  • This pull request modifies code in src/* but no tests were added/updated.
    • Confirm whether tests should be added or ensure the PR description explains why tests are not required.

@willyyhuang willyyhuang force-pushed the wlsun/add-auto-build-feature-opt-in-out branch from d987ed4 to 07d2d3f Compare November 6, 2024 15:33
@willyyhuang willyyhuang changed the title feat(feature dev): Add setting to opt in/out of QSDA auto build feature feat(feature dev): Add setting to allow Q to run code and test commands Nov 6, 2024
@willyyhuang willyyhuang marked this pull request as ready for review November 6, 2024 16:00
@willyyhuang willyyhuang requested review from a team as code owners November 6, 2024 16:00
@willyyhuang willyyhuang force-pushed the wlsun/add-auto-build-feature-opt-in-out branch 2 times, most recently from 9c5a9d8 to f1cf313 Compare November 6, 2024 19:05
@willyyhuang willyyhuang force-pushed the wlsun/add-auto-build-feature-opt-in-out branch from f1cf313 to dc3b7e6 Compare November 6, 2024 20:50
@willyyhuang willyyhuang force-pushed the wlsun/add-auto-build-feature-opt-in-out branch 2 times, most recently from 433a97d to 3faa157 Compare November 7, 2024 18:21
@willyyhuang willyyhuang force-pushed the wlsun/add-auto-build-feature-opt-in-out branch from 3faa157 to 3e73034 Compare November 7, 2024 18:48
@jpinkney-aws jpinkney-aws merged commit e23c61d into aws:feature/q-dev-execution Nov 12, 2024
20 of 25 checks passed
jpinkney-aws pushed a commit that referenced this pull request Nov 14, 2024
…ing (#5980)

## Problem

Follow up PR from #5853 

## Solution

Added more unit test coverage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants