Skip to content

Conversation

@jacob-local-kevin
Copy link
Contributor

Changes Performed:

Step 1: Add skipBuild flag to RunBuildCheckParams

Files:

/src/server/build/node/check.ts

Details:

Modify the RunBuildCheckParams interface to include a new optional boolean property named skipBuild. This will allow the runBuildCheck function to conditionally skip the build process based on this flag.

Exit Criteria

The RunBuildCheckParams interface includes a skipBuild flag, and the runBuildCheck function can conditionally skip the build process based on this flag.

Step 2: Extract skipBuild flag from GitHub issue text

Files:

/src/server/messaging/queue.ts

Details:

Update the onGitHubEvent function to extract the --skip-build flag from the GitHub issue text. Add this flag to the baseEventData object, which is used throughout the application to pass event-related data.

Exit Criteria

The onGitHubEvent function extracts the --skip-build flag from the issue text and includes it in the baseEventData object.

Step 3: Pass skipBuild flag in createTodo function

Files:

/src/server/utils/todos.ts

Details:

Modify the createTodo function to pass the skipBuild flag to subsequent functions if it is present in the issue text. This will ensure that the flag is propagated correctly throughout the application.

Exit Criteria

The createTodo function passes the skipBuild flag to subsequent functions if present in the issue text.

Step 4: Include skipBuild flag in extracted issue information

Files:

/src/server/api/issues.ts

Details:

Update the getExtractedIssues function to include the skipBuild flag in the extracted issue information. This will ensure that the flag is available when processing issues.

Exit Criteria

The getExtractedIssues function includes the skipBuild flag in the extracted issue information.

Step 5: Add skipBuild field to ExtractedIssueInfoSchema

Files:

/src/server/code/extractedIssue.ts

Details:

Modify the ExtractedIssueInfoSchema to include a field for the skipBuild flag. This will allow the schema to validate the presence of the flag in extracted issue information.

Exit Criteria

The ExtractedIssueInfoSchema includes a field for the skipBuild flag.

@jacob-local-kevin
Copy link
Contributor Author

Hello human! 👋

This PR was created by JACoB to address the issue Add Skip Build functionality

Next Steps

  1. Please review the PR carefully. Auto-generated code can and will contain subtle bugs and mistakes.

  2. If you identify code that needs to be changed, please reject the PR with a specific reason.
    Be as detailed as possible in your comments. JACoB will take these comments, make changes to the code and push up changes.
    Please note that this process will take a few minutes.

  3. Once the code looks good, approve the PR and merge the code.

kleneway added a commit that referenced this pull request Oct 1, 2024
added backup for GROQ_API_KEY for build
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.

1 participant