Skip to content

Update logo in README and fix package version parameter in publish workflow#43

Merged
shibayan merged 2 commits into
masterfrom
fix-version
Apr 12, 2026
Merged

Update logo in README and fix package version parameter in publish workflow#43
shibayan merged 2 commits into
masterfrom
fix-version

Conversation

@shibayan

Copy link
Copy Markdown
Owner

This pull request makes minor updates to improve project documentation and corrects a property used in the build workflow.

  • Documentation update:

    • Changed the logo in README.md from an SVG to a PNG to improve compatibility or display.
  • Build workflow correction:

    • In .github/workflows/publish.yml, updated the property passed to the build command from PackageVersion to Version for consistency with expected parameters.

@shibayan shibayan requested a review from Copilot April 12, 2026 04:42
@shibayan shibayan self-assigned this Apr 12, 2026
@shibayan shibayan added bug Something isn't working documentation Improvements or additions to documentation labels Apr 12, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 makes small documentation and release workflow adjustments for OpenApiWeaver: it updates the README logo asset to a PNG and tweaks the NuGet publish workflow’s MSBuild property used for versioning during packing.

Changes:

  • Update README.md to reference a PNG logo instead of the SVG.
  • Add docs/public/logo.png to the docs/public assets.
  • Update the NuGet publish workflow to pass -p:Version=... during dotnet pack.

Reviewed changes

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

File Description
README.md Switches the README logo reference from SVG to PNG.
docs/public/logo.png Adds the PNG logo asset used by the README.
.github/workflows/publish.yml Updates the MSBuild property passed to dotnet pack for versioning.

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

Comment thread .github/workflows/publish.yml
@shibayan shibayan merged commit ce3c4b9 into master Apr 12, 2026
4 checks passed
@shibayan shibayan deleted the fix-version branch April 12, 2026 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants