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

Jump to bottom

Fix bug with vanilla level costs and remove v prefix from release versions #10

Merged
merged 4 commits into from
May 26, 2024
Merged

Fix bug with vanilla level costs and remove v prefix from release versions #10

merged 4 commits into from
May 26, 2024

Conversation

zefir-git
Copy link
Member

  • Cost is not added anymore if the level is within the vanilla limits
  • The release workflow has been updated to strip the v prefix of tags when setting the release version
  • The readme received minor improvements

@zefir-git zefir-git added bug Something isn't working enhancement Improvement of existing feature labels May 26, 2024
@zefir-git zefir-git requested a review from a team May 26, 2024 11:05
@zefir-git zefir-git self-assigned this May 26, 2024
@zefir-git zefir-git enabled auto-merge May 26, 2024 11:13
kai-gitt
kai-gitt approved these changes May 26, 2024
View reviewed changes
Copy link
Member

@kai-gitt kai-gitt left a comment

Choose a reason for hiding this comment

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

lgtm

@zefir-git zefir-git merged commit c98115f into main May 26, 2024
1 check passed
@zefir-git zefir-git deleted the fixes branch May 26, 2024 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kai-gitt kai-gitt kai-gitt approved these changes

Assignees

@zefir-git zefir-git

Labels
bug Something isn't working enhancement Improvement of existing feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zefir-git @kai-gitt