Skip to content

Fix build failure: downgrade Microsoft.Graph to available version and remove duplicate test file#3

Merged
evilz merged 2 commits intomasterfrom
copilot/fix-build-and-ci-issues
Feb 5, 2026
Merged

Fix build failure: downgrade Microsoft.Graph to available version and remove duplicate test file#3
evilz merged 2 commits intomasterfrom
copilot/fix-build-and-ci-issues

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 5, 2026

Build was broken due to Microsoft.Graph 6.0.0 not existing and duplicate test class definitions causing compilation errors.

Changes

  • Downgrade Microsoft.Graph from 6.0.0 to 5.101.0 in Todo.Core.csproj

    • Version 6.0.0 does not exist in nuget.org (latest available is 5.101.0)
    • This was blocking dotnet restore
  • Remove src/Todo.CLI.Tests/Commands/CommandTests.cs

    • Contained duplicate definitions of AddCommandTests, ListCommandTests, CompleteCommandTests, and RemoveCommandTests
    • Individual files for these test classes already exist with more comprehensive tests
    • Duplicates caused CS0101 compilation errors

Build now succeeds with all 43 tests passing. Package publishing verified for linux-x64, win-x64, and osx-x64 targets.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • login.microsoftonline.com
    • Triggering command: /tmp/publish-test/linux-x64/todo /tmp/publish-test/linux-x64/todo --help (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Fix the build and CI, it's not working.
Keep doing the work until everything is green and package can be released


💡 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.

…t file

Co-authored-by: evilz <2937862+evilz@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix build and CI to ensure successful releases Fix build failure: downgrade Microsoft.Graph to available version and remove duplicate test file Feb 5, 2026
Copilot AI requested a review from evilz February 5, 2026 09:22
@evilz evilz marked this pull request as ready for review February 5, 2026 09:33
Copilot AI review requested due to automatic review settings February 5, 2026 09:33
@evilz evilz merged commit 0c5eea7 into master Feb 5, 2026
5 checks passed
Copy link
Copy Markdown

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 fixes build failures caused by a non-existent package version and duplicate test class definitions. The changes restore the ability to build and test the project successfully.

Changes:

  • Downgraded Microsoft.Graph from 6.0.0 to 5.101.0 (the latest available version)
  • Removed duplicate test file that conflicted with existing comprehensive test files

Reviewed changes

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

File Description
src/Todo.Core/Todo.Core.csproj Corrected Microsoft.Graph package version from non-existent 6.0.0 to available 5.101.0
src/Todo.CLI.Tests/Commands/CommandTests.cs Removed duplicate test class definitions that conflicted with existing comprehensive test files

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

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