Skip to content

Conversation

@ebajrami
Copy link
Collaborator

Description

Update for the PR #177

Type of Change

Since publishing can be only triggered based on the git tag, we are adding two things here:

  • 🚀 New feature (non-breaking change)
  • 🛠️ Bug fix (non-breaking change)
  • ⚠️ Breaking change (feature or bug fix which breaks existing behaviors/APIs)
  • 🏗️ Code refactor
  • ⚙️ Build configuration change
  • 📝 Documentation
  • 🧹 Chore / Housekeeping

@ebajrami ebajrami added the ready for pipeline Attach this label to your PR to make the pipeline run the action label May 22, 2025
@ebajrami ebajrami force-pushed the ebajrami/issue-176 branch 2 times, most recently from 6f022b5 to efa1f0b Compare May 22, 2025 16:35
@ebajrami ebajrami force-pushed the ebajrami/issue-176 branch from efa1f0b to 6600e1d Compare May 22, 2025 16:38
4ndrey
4ndrey previously approved these changes May 23, 2025
Copy link
Collaborator

@dominik089 dominik089 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left some comments (mostly linting) regarding the shellscripts

@ebajrami ebajrami force-pushed the ebajrami/issue-176 branch 3 times, most recently from e78d1a1 to 91bf060 Compare May 23, 2025 16:00
Copy link
Collaborator

@robert-northmind robert-northmind left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM if you fix Dominik's comments.
Nice work 🥳

@ebajrami ebajrami force-pushed the ebajrami/issue-176 branch 5 times, most recently from 52d4abe to 3e66a41 Compare May 26, 2025 11:53
@ebajrami ebajrami force-pushed the ebajrami/issue-176 branch 4 times, most recently from 7ed7091 to 18f1eab Compare May 26, 2025 13:12
@ebajrami ebajrami merged commit 76e756d into master May 26, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for pipeline Attach this label to your PR to make the pipeline run the action

Development

Successfully merging this pull request may close these issues.

5 participants