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

rysweet-4677-rename-agents-project-to-core-trim-dependencies #4696

Merged

Conversation

rysweet
Copy link
Collaborator

@rysweet rysweet commented Dec 13, 2024

  • renames Agents to Core
  • New project Agents just includes base agents
  • Core should just depend on Contracts

Why are these changes needed?

  • improves dependencies and makes tests/refactor easier

Related issue number

closes #4677

Checks

@rysweet rysweet added the dotnet issues related to AutoGen.Net label Dec 13, 2024
@rysweet rysweet added this to the 0.4.0 milestone Dec 13, 2024
@rysweet rysweet marked this pull request as draft December 13, 2024 17:27

Choose a reason for hiding this comment

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

Copilot reviewed 36 out of 51 changed files in this pull request and generated no comments.

Files not reviewed (15)
  • dotnet/AutoGen.sln: Language not supported
  • dotnet/samples/Hello/HelloAIAgents/HelloAIAgents.csproj: Language not supported
  • dotnet/samples/Hello/HelloAgent/HelloAgent.csproj: Language not supported
  • dotnet/samples/Hello/HelloAgentState/HelloAgentState.csproj: Language not supported
  • dotnet/samples/dev-team/DevTeam.AgentHost/DevTeam.AgentHost.csproj: Language not supported
  • dotnet/samples/dev-team/DevTeam.Agents/DevTeam.Agents.csproj: Language not supported
  • dotnet/samples/dev-team/DevTeam.Backend/DevTeam.Backend.csproj: Language not supported
  • dotnet/samples/dev-team/DevTeam.Agents/DeveloperLead/DeveloperLead.cs: Evaluated as low risk
  • dotnet/samples/Hello/HelloAgentState/Program.cs: Evaluated as low risk
  • dotnet/samples/dev-team/DevTeam.Agents/ProductManager/ProductManager.cs: Evaluated as low risk
  • dotnet/samples/dev-team/DevTeam.Agents/Developer/Developer.cs: Evaluated as low risk
  • dotnet/samples/dev-team/DevTeam.Backend/Agents/Hubber.cs: Evaluated as low risk
  • dotnet/samples/dev-team/DevTeam.Backend/Program.cs: Evaluated as low risk
  • dotnet/samples/dev-team/DevTeam.Backend/Agents/AzureGenie.cs: Evaluated as low risk
  • dotnet/samples/dev-team/DevTeam.Backend/Services/GithubWebHookProcessor.cs: Evaluated as low risk
Base automatically changed from rysweet-4688-core-doesnt-use-orleans to main December 13, 2024 18:32
@rysweet rysweet marked this pull request as ready for review December 13, 2024 19:39
@rysweet rysweet merged commit a19c848 into main Dec 13, 2024
49 checks passed
@rysweet rysweet linked an issue Dec 13, 2024 that may be closed by this pull request
@rysweet rysweet deleted the 4677-rename-agents-package---core-which-depends-only-on-contracts branch December 13, 2024 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dotnet issues related to AutoGen.Net
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rename Agents package -> Core which depends only on contracts
2 participants