Skip to content

Conversation

Copy link

Copilot AI commented Dec 6, 2025

Description

Increments package version and adds shields to README for package discoverability. Also fixes markdown formatting issues in the Azure Cosmos DB Setup section.

Related Issues

Addresses version bump and documentation enhancement request.

Proposed Changes

  • Version: 10.0.6 → 10.0.7 in Directory.Build.props
  • README badges: Added shields.io badges for:
    • All 5 NuGet packages (ManagedCode.GraphRag, Neo4j, Postgres, CosmosDb, JanusGraph)
    • GitHub Actions CI status
    • MIT License
  • Markdown formatting: Fixed code block indentation in Azure Cosmos DB Setup section to properly align with numbered list items

Badges positioned immediately after title for visibility. Auto-update from shields.io as packages publish.

Checklist

  • I have tested these changes locally.
  • I have reviewed the code changes.
  • I have updated the documentation (if necessary).
  • I have added appropriate unit tests (if applicable).

Additional Notes

Minor formatting applied by dotnet format included for consistency.

Original prompt

This section details on the original issue you should resolve

<issue_title>Bump version</issue_title>
<issue_description>Bumpt version and add into Readme shields for nuget packages and so on </issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: KSemenenko <4385716+KSemenenko@users.noreply.github.com>
@KSemenenko
Copy link
Member

@copilot chedk in read my Azure Cosmos DB Setup

The Cosmos adapter (ManagedCode.GraphRag.CosmosDb) targets the SQL API and works with the emulator or live accounts:

Provide a connection string. Set COSMOS_EMULATOR_CONNECTION_STRING or configure options manually.
Register the store.
builder.Services.AddCosmosGraphStore("cosmos", options => { options.ConnectionString = cosmosConnectionString; options.DatabaseId = "GraphRagIntegration"; options.NodesContainerId = "nodes"; options.EdgesContainerId = "edges"; options.ConfigureClientOptions = clientOptions => { clientOptions.GatewayModeMaxConnectionLimit = 100; }; options.ConfigureSerializer = serializer => serializer.PropertyNamingPolicy = null; }); I think this is broke markdown fix it

Copilot AI changed the title [WIP] Bump version and add NuGet shields to README Bump version to 10.0.7 and add NuGet package badges Dec 6, 2025
Copilot AI requested a review from KSemenenko December 6, 2025 08:14
Co-authored-by: KSemenenko <4385716+KSemenenko@users.noreply.github.com>
Copy link
Author

Copilot AI commented Dec 6, 2025

@copilot chedk in read my Azure Cosmos DB Setup

The Cosmos adapter (ManagedCode.GraphRag.CosmosDb) targets the SQL API and works with the emulator or live accounts:
...

Fixed in c84a1a4. The code block is now properly indented (3 spaces) to align with the numbered list item, ensuring correct markdown rendering.

@KSemenenko KSemenenko marked this pull request as ready for review December 6, 2025 08:52
Copilot AI review requested due to automatic review settings December 6, 2025 08:52
@KSemenenko KSemenenko merged commit 56b6389 into main Dec 6, 2025
4 checks passed
@KSemenenko KSemenenko deleted the copilot/bump-version-and-update-readme branch December 6, 2025 08:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR bumps the package version from 10.0.6 to 10.0.7 and enhances the README with NuGet package badges and CI status indicators. It also includes minor code formatting improvements applied by dotnet format.

Key Changes:

  • Incremented package version to 10.0.7 across all NuGet packages
  • Added shields.io badges for all 5 NuGet packages (ManagedCode.GraphRag, Neo4j, Postgres, CosmosDb, JanusGraph), GitHub Actions CI status, and MIT license
  • Fixed markdown code block indentation in the Azure Cosmos DB Setup section to properly render within the numbered list

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
Directory.Build.props Incremented Version and PackageVersion from 10.0.6 to 10.0.7
README.md Added NuGet, CI, and license badges; fixed code block indentation in Azure Cosmos DB Setup section
tests/ManagedCode.GraphRag.Tests/Storage/Postgres/PostgresAgtypeParameterTests.cs Reordered imports alphabetically (formatting)
src/ManagedCode.GraphRag/Utils/Hashing.cs Expanded single-line if statement to multi-line format (formatting)
src/ManagedCode.GraphRag.Postgres/ApacheAge/AgeConnectionManager.cs Condensed null check into inline throw expression (formatting)
Comments suppressed due to low confidence (1)

src/ManagedCode.GraphRag.Postgres/ApacheAge/AgeConnectionManager.cs:158

  • Poor error handling: empty catch block.
            catch (PostgresException)
            {
            }

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

{
throw new AgeException("AGE extension is not installed.");
}
var result = await checkCommand.ExecuteScalarAsync(cancellationToken).ConfigureAwait(false) ?? throw new AgeException("AGE extension is not installed.");
Copy link

Copilot AI Dec 6, 2025

Choose a reason for hiding this comment

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

This assignment to result is useless, since its value is never read.

Suggested change
var result = await checkCommand.ExecuteScalarAsync(cancellationToken).ConfigureAwait(false) ?? throw new AgeException("AGE extension is not installed.");
await checkCommand.ExecuteScalarAsync(cancellationToken).ConfigureAwait(false) ?? throw new AgeException("AGE extension is not installed.");

Copilot uses AI. Check for mistakes.
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.

Bump version

2 participants