Skip to content
This repository was archived by the owner on Jul 21, 2025. It is now read-only.

Conversation

@aymanaboghonim
Copy link

@aymanaboghonim aymanaboghonim commented Jul 1, 2025

Summary
Improves the documentation in docs/RunApplication.md with specific repository details and clearer instructions.

Changes Made

  • Added explicit repository name to header: "Running the AzDevOpsDemoGenerator Application"
  • Fixed clone command with actual GitHub URL instead of placeholders
  • Specified exact solution file path (src/ADOGenerator.sln) for clarity
  • Surgical changes with no formatting modifications to existing content

Why These Changes

  • Repository name: Makes it clear which application is being documented
  • Real clone URL: Users can copy-paste the actual command instead of replacing placeholders
  • Exact file path: Eliminates confusion about where to find the solution file

Testing

  • Verified all paths and URLs are correct
  • No structural or formatting changes to existing content
  • Instructions remain clear and actionable

Type of Change

  • Documentation improvement
  • User experience enhancement
  • No breaking changes

Related Issues
Fixes part of #66 - specifically addresses Problem 3 (docs/RunApplication.md improvements)

- Add explicit repository name to header (AzDevOpsDemoGenerator)
- Fix clone command with actual GitHub URL
- Specify exact solution file path (src/ADOGenerator.sln)
- Surgical changes with no formatting modifications
Copilot AI review requested due to automatic review settings July 1, 2025 10:21
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

Updates the RunApplication.md docs to use the real repository name, clone URL, and solution path for clearer setup instructions.

  • Added explicit repository name in the header
  • Replaced placeholder clone command with actual GitHub URL and directory name
  • Specified exact solution file path under src/
Comments suppressed due to low confidence (3)

docs/RunApplication.md:1

  • [nitpick] The project name AzDevOpsDemoGenerator is presented as a single token; consider adding spaces for readability (e.g., Az DevOps Demo Generator) or matching the exact branding used elsewhere in the repo.
# Running the AzDevOpsDemoGenerator Application

docs/RunApplication.md:3

  • The header refers to AzDevOpsDemoGenerator but this line uses ADOGenerator; consider clarifying that the repository is named AzDevOpsDemoGenerator while the solution/project is ADOGenerator to avoid confusion.
To run the ADOGenerator project as a console application or executable, follow these steps:

docs/RunApplication.md:16

  • [nitpick] Commands inside the fenced code block are indented by spaces; removing the leading spaces will align them to the left margin and improve readability in the rendered doc.
   git clone https://github.com/microsoft/AzDevOpsDemoGenerator.git

@Ildrim7
Copy link

Ildrim7 commented Jul 4, 2025

It says Debug profile doesn't exists and there 16 errors in this code. Am I the only one facing this issue
error

@aymanaboghonim
Copy link
Author

It worked fine with me!

@Ildrim7
Copy link

Ildrim7 commented Jul 4, 2025

Unable to find package log4net. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
Unable to find package System.Threading. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
Unable to find package Newtonsoft.Json with version (>= 13.0.3)

  • Found 1 version(s) in Microsoft Visual Studio Offline Packages [ Nearest version: 13.0.1 ]
    Unable to find package System.Formats.Asn1. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package Microsoft.Extensions.Configuration.Json. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package System.Text.RegularExpressions. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package Microsoft.Extensions.Configuration.FileExtensions. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package Microsoft.Identity.Client. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package System.Net.Http. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package Microsoft.VisualStudio.Services.Client. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package Microsoft.VisualStudio.Services.ExtensionManagement.WebApi. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package System.Data.SqlClient. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package Microsoft.Extensions.Configuration. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package log4net. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package System.Threading. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
    Unable to find package Newtonsoft.Json with version (>= 13.0.3)
  • Found 1 version(s) in Microsoft Visual Studio Offline Packages [ Nearest version: 13.0.1 ]

THESE ARE THE ERRORS

@Ildrim7
Copy link

Ildrim7 commented Jul 4, 2025

There's no ADO exe file as well in the code

@Ildrim7
Copy link

Ildrim7 commented Jul 4, 2025

The issue was with NuGet Package Manager in Visual Studio. It was trying to take the offline package sources. I change the package source https://api.nuget.org/v3/index.jason. Then it did all the steps again.

It resolved the issue. Thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants