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.

Sign up for GitHub

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

Fix nuget package #474

Merged
merged 2 commits into from
May 4, 2024
Merged

Fix nuget package #474

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4f823ee
fix readme and add it to nuget package
laedit May 4, 2024
de2e21c
fix sonar warning
laedit May 4, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/sonar-analysis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,6 @@ jobs:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
run: |
./.sonar/scanner/dotnet-sonarscanner begin /k:"laedit_vika" /o:"laedit-github" /d:sonar.login="${{ secrets.SONAR_TOKEN }}" /d:sonar.host.url="https://sonarcloud.io" /d:sonar.cs.opencover.reportsPaths=src/NVika.Tests/coverage.opencover.xml /d:sonar.coverage.exclusions="src/NVika.Tests/*"
./.sonar/scanner/dotnet-sonarscanner begin /k:"laedit_vika" /o:"laedit-github" /d:sonar.token="${{ secrets.SONAR_TOKEN }}" /d:sonar.host.url="https://sonarcloud.io" /d:sonar.cs.opencover.reportsPaths=src/NVika.Tests/coverage.opencover.xml /d:sonar.coverage.exclusions="src/NVika.Tests/*"
dotnet build src --configuration Release /p:Version=${{ steps.gitversion.outputs.nuGetVersionV2 }}
./.sonar/scanner/dotnet-sonarscanner end /d:sonar.login="${{ secrets.SONAR_TOKEN }}"
./.sonar/scanner/dotnet-sonarscanner end /d:sonar.token="${{ secrets.SONAR_TOKEN }}"
18 changes: 5 additions & 13 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,7 @@ Also means 'bug' in Finnish.

[![Licence Apache 2](https://img.shields.io/badge/licence-Apache%202-blue.svg)](https://github.com/laedit/vika/blob/master/LICENSE)
![Build](https://github.com/laedit/vika/workflows/Build/badge.svg)
[![Sonarcloud Status](https://sonarcloud.io/api/project_badges/measure?project=laedit_vika&metric=alert_status)](https://sonarcloud.io/dashboard?id=laedit_vika)
[![SonarCloud Coverage](https://sonarcloud.io/api/project_badges/measure?project=laedit_vika&metric=coverage)](https://sonarcloud.io/component_measures/metric/coverage/list?id=laedit_vika)
[![SonarCloud Bugs](https://sonarcloud.io/api/project_badges/measure?project=laedit_vika&metric=bugs)](https://sonarcloud.io/component_measures/metric/reliability_rating/list?id=laedit_vika)
[![SonarCloud Vulnerabilities](https://sonarcloud.io/api/project_badges/measure?project=laedit_vika&metric=vulnerabilities)](https://sonarcloud.io/component_measures/metric/security_rating/list?id=laedit_vika)
[![Sonarcloud Status](https://sonarcloud.io/api/project_badges/measure?project=laedit_vika&metric=alert_status)](https://sonarcloud.io/project/overview?id=laedit_vika)

## What it is
Right now it's just a tiny tool which parse analysis reports and send messages to the build server, or in console if it's not executed on a build server.
Expand Down Expand Up @@ -56,9 +53,11 @@ It is possible to process several reports at the same time: `NVika parsereport r

## Build servers
### Supported
- [AppVeyor](http://appveyor.com)
- Local build: output in console
- [AppVeyor](https://appveyor.com)
![AppVeyor example](resources/AppVeyor.png)

- [GitHub actions](https://github.com)

### To come
- [TeamCity](https://github.com/laedit/vika/issues/4)?
- [ContinuaCI](https://github.com/laedit/vika/issues/3)?
Expand All @@ -67,13 +66,6 @@ It is possible to process several reports at the same time: `NVika parsereport r
I am really wondering if there is any value to supporting these three, because there doesn't support to add build message like AppVeyor but only log message.
And they support custom HTML report, so an xslt transformation is enough.

## What it will be
A website will be added for displaying a nice and shiny aggregated report from all source to a dedicated page for each GitHub project.

There also will be a solution to upload a temporary report stored for a week.

And the client may push reports through the website public API.

## Contributing
All contributions are welcome, please read our [contributing guide](CONTRIBUTING.md).

Expand Down
12 changes: 4 additions & 8 deletions src/NVika/NVika.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
<RepositoryType>git</RepositoryType>
<PackageTags>report parsing build server inspectcode FxCop SARIF Roslyn Gendarme</PackageTags>
<PackageReleaseNotes>https://github.com/laedit/vika/releases</PackageReleaseNotes>
<PackageReadmeFile>README.md</PackageReadmeFile>
<AnalysisMode>AllEnabledByDefault</AnalysisMode>
<Deterministic>true</Deterministic>
</PropertyGroup>
Expand Down Expand Up @@ -43,14 +44,9 @@
</ItemGroup>

<ItemGroup>
<None Include="..\..\resources\icon.png">
<Pack>True</Pack>
<PackagePath></PackagePath>
</None>
<None Include="..\..\LICENSE">
<Pack>True</Pack>
<PackagePath></PackagePath>
</None>
<None Include="..\..\resources\icon.png" Pack="true" PackagePath="\" />
<None Include="..\..\LICENSE" Pack="true" PackagePath="\" />
<None Include="..\..\README.md" Pack="true" PackagePath="\" />
</ItemGroup>

<ItemGroup>
Expand Down
Loading