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.

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

Update state logic #23

Merged
merged 5 commits into from
Feb 22, 2022
Merged

Update state logic #23

merged 5 commits into from
Feb 22, 2022

Conversation

babarot
Copy link
Owner

@babarot babarot commented Feb 22, 2022

WHAT

  • Use state.ID instead of state.Name
  • Add state rm command
  • Add annotation feature related to bump up version

WHY

Currently in state file, each state name is same as pkg name.
But if using pkg.name as state.name, it may be problem:

  • changing pkg.name in config file by users

this operation will be regarded as adding new package and
removing that package.

so we need fixed string not depending on pkg.name

Currently in state file, each state name is same as pkg name.
But if using pkg.name as state.name, it may be problem:

- changing pkg.name in config file by users

this operation will be regarded as adding new package and
removing that package.

so we need fixed string not depending on pkg.name
@babarot babarot added the release/patch Indicates patch update to action-release-label label Feb 22, 2022
@github-actions
Copy link

This PR will update b4b4r07/afx from v0.1.10 to v0.1.11 🚀

Changes: v0.1.10...feature/state

If this update isn't as you expected, you may want to change or remove the release label.

@babarot babarot merged commit efe7664 into master Feb 22, 2022
@babarot babarot deleted the feature/state branch February 22, 2022 14:37
@github-actions
Copy link

The new version v0.1.11 has been released 🎉

Changes: v0.1.10...v0.1.11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/patch Indicates patch update to action-release-label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant