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: reimplement add comment to issues + prs and generate useful content #1451

Closed
wants to merge 218 commits into from

Conversation

monilpat
Copy link
Collaborator

Relates to:

Risks

Low. The changes involve updating methods to retrieve specific pull request and issue numbers, and ensuring types and schemas are correctly defined. This could affect the functionality of adding comments to pull requests and issues if not implemented correctly.

Background

What does this PR do?

This PR updates the methods used to retrieve specific pull request and issue numbers in the addCommentToPR.ts and addCommentToIssue.ts files. It also ensures that types and schemas are correctly defined and used in the types.ts file.

What kind of change is this?

  • Improvements (misc. changes to existing features)

Documentation changes needed?

My changes do not require a change to the project documentation.

Testing

Where should a reviewer start?

Start by reviewing the changes in the addCommentToPR.ts, addCommentToIssue.ts, and types.ts files.

Detailed testing steps

  • Verify that comments can be successfully added to pull requests and issues using the updated methods.
  • Ensure that the types and schemas are correctly validated.

Linter Errors

File Name: packages/plugin-github/src/types.ts
Errors:


1 |


monilpat and others added 30 commits November 13, 2024 22:08
…apters, and Multi-Agent Ecosystem Goals - 11.24.24
…apters, and Multi-Agent Ecosystem Goals - 11.24.24
…apters, and Multi-Agent Ecosystem Goals - 11.24.24 to Work Log 11.24.24 - Reality Spiraling: Defining AI-Driven Narratives, Technical Adapters, and Multi-Agent Ecosystem Goals
…atives, Technical Adapters, and Multi-Agent Ecosystem Goals to Work Log - 11.24.24 - Reality Spiraling: Defining AI-Driven Narratives, Technical Adapters, and Multi-Agent Ecosystem Goals
@monilpat monilpat closed this Dec 25, 2024
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.

7 participants