Skip to content

chore: move icon to assets/ to fix release-plz#177

Merged
zeitlinger merged 1 commit intomainfrom
fix/idea-gitignore
Apr 16, 2026
Merged

chore: move icon to assets/ to fix release-plz#177
zeitlinger merged 1 commit intomainfrom
fix/idea-gitignore

Conversation

@zeitlinger
Copy link
Copy Markdown
Member

Summary

  • .idea is in .gitignore but icon.svg was committed there
  • release-plz bails with "uncommitted changes" on every push to main (run)
  • Move to assets/icon.svg and update README reference

Test plan

  • Merge and verify release-plz "Release PR" job succeeds on next push to main

.idea is in .gitignore but icon.svg was committed there, causing
release-plz to bail with "uncommitted changes" on every push to main.
Move to assets/icon.svg and update README reference.
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@zeitlinger zeitlinger marked this pull request as ready for review April 16, 2026 14:31
@zeitlinger zeitlinger requested a review from a team as a code owner April 16, 2026 14:31
Copilot AI review requested due to automatic review settings April 16, 2026 14:31
@zeitlinger zeitlinger enabled auto-merge (squash) April 16, 2026 14:31
@zeitlinger zeitlinger merged commit fb651bd into main Apr 16, 2026
16 checks passed
@zeitlinger zeitlinger deleted the fix/idea-gitignore branch April 16, 2026 14:34
Copy link
Copy Markdown
Contributor

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

Moves the project logo out of the ignored .idea/ directory into a tracked assets/ directory to prevent release-plz from detecting a dirty working tree, and updates documentation to reference the new location.

Changes:

  • Add assets/icon.svg as the new tracked icon location.
  • Update README.md to reference assets/icon.svg instead of .idea/icon.svg.

Reviewed changes

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

File Description
assets/icon.svg Introduces a new committed icon file under assets/.
README.md Updates the README logo image path to point at the new asset location.

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

@github-actions github-actions bot mentioned this pull request Apr 16, 2026
zeitlinger added a commit that referenced this pull request Apr 16, 2026
### Added

- *(release)* migrate from release-please to release-plz
([#171](#171))

### Fixed

- *(release)* use correct template variable in pr_body
([#178](#178))
- *(release)* suppress component prefix in release-please tags
([#166](#166))
- *(release)* add workflow_dispatch to retrigger for existing tags
([#167](#167))

### Other

- move icon to assets/ to fix release-plz
([#177](#177))
- *(deps)* update dependency npm:renovate to v43.102.11 [security]
([#174](#174))
- *(deps)* update rust crate similar to v3.1.0
([#173](#173))
- *(deps)* update dependency github:mvdan/sh to v3.13.1
([#163](#163))

> [!IMPORTANT]
> Close and reopen this PR to trigger CI checks.

---------

Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
This was referenced Apr 16, 2026
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