Skip to content

XRAY-135041 slnx support added for dotnet - #808

Open
Phavya-jfrog wants to merge 4 commits into
jfrog:devfrom
Phavya-jfrog:feature/XRAY-135041-slnx
Open

XRAY-135041 slnx support added for dotnet#808
Phavya-jfrog wants to merge 4 commits into
jfrog:devfrom
Phavya-jfrog:feature/XRAY-135041-slnx

Conversation

@Phavya-jfrog

@Phavya-jfrog Phavya-jfrog commented Jul 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • The pull request is targeting the dev branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....
  • All static analysis checks passed.
  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • Updated the Contributing page / ReadMe page / CI Workflow files if needed.
  • All changes are detailed at the description. if not already covered at JFrog Documentation, new documentation have been added.

jf curation-audit silently skipped .NET projects with only a .slnx file, since technology detection never recognized it.

  • Added .slnx to Nuget/Dotnet indicators and package descriptors.
  • Fixed --solution-path not being passed into solution.Load() (was hardcoded to ""), which made solution.Load() auto-discover and merge projects from every solution file instead of honoring the explicit flag.
  • Updated --solution-path flag description to mention .slnx.
Screenshot 2026-07-24 at 11 34 51 AM

@Phavya-jfrog
Phavya-jfrog force-pushed the feature/XRAY-135041-slnx branch from cf310a6 to 2fde89d Compare July 23, 2026 12:36
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Jul 24, 2026
gineshkumar
gineshkumar reviewed Jul 27, 2026
View reviewed changes
Comment thread utils/techutils/techutils.go
gineshkumar
gineshkumar reviewed Jul 27, 2026
View reviewed changes
Comment thread sca/bom/buildinfo/technologies/nuget/nuget_test.go
gineshkumar
gineshkumar reviewed Jul 27, 2026
View reviewed changes
Comment thread sca/bom/buildinfo/technologies/nuget/nuget.go
gineshkumar
gineshkumar reviewed Jul 27, 2026
View reviewed changes
Comment thread sca/bom/buildinfo/technologies/nuget/nuget.go
gineshkumar
gineshkumar reviewed Jul 27, 2026
View reviewed changes
Comment thread sca/bom/buildinfo/technologies/nuget/nuget_test.go Outdated
@Phavya-jfrog Phavya-jfrog added safe to test Approve running integration tests on a pull request and removed safe to test Approve running integration tests on a pull request labels Jul 27, 2026
@Phavya-jfrog
Phavya-jfrog force-pushed the feature/XRAY-135041-slnx branch from a0b9b3a to 119ac2c Compare July 27, 2026 12:02
@Phavya-jfrog
Phavya-jfrog force-pushed the feature/XRAY-135041-slnx branch from bcda04a to 599fe8c Compare July 28, 2026 06:24
@Phavya-jfrog Phavya-jfrog added safe to test Approve running integration tests on a pull request and removed safe to test Approve running integration tests on a pull request labels Jul 28, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Jul 28, 2026
@Phavya-jfrog
Phavya-jfrog force-pushed the feature/XRAY-135041-slnx branch from e80d4eb to e369177 Compare July 28, 2026 10:20
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Jul 28, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Jul 28, 2026
@Phavya-jfrog
Phavya-jfrog force-pushed the feature/XRAY-135041-slnx branch from f8481d5 to ba664e6 Compare July 28, 2026 13:23
@Phavya-jfrog Phavya-jfrog added improvement Automatically generated release notes safe to test Approve running integration tests on a pull request labels Jul 28, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Jul 28, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Jul 28, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Jul 28, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Jul 28, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Jul 28, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Jul 29, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@gineshkumar gineshkumar gineshkumar left review comments
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

improvement Automatically generated release notes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Phavya-jfrog @gineshkumar