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

Feat/hooks #51

Merged
merged 4 commits into from
Nov 8, 2024
Merged

Feat/hooks #51

merged 4 commits into from
Nov 8, 2024

Conversation

sergiotejon
Copy link
Contributor

Improve handling of empty fields when running the get command to avoid printing omitempy tags, also.

… in ConfigVersion struct

- Updated loop logic to handle cases when fields are empty
- Removed unnecessary code block for handling reflected type
- Added function getAllFieldNames() to retrieve all field names of a given type
- Modified loop to iterate over all fields and assign values based on their presence in the reflected value
- Implemented check to ensure that the field exists before proceeding with assignment
- Updated test cases to cover the changes made

This commit refactors the handling of empty fields in the ConfigVersion struct, improving its functionality and readability. The modifications also increase the maintainability of the code by eliminating unnecessary repetition and adding a helper function for retrieving all field names of a given type.
…ost-changelog script execution and changelog generation

* Add handling for running pre-changelog scripts before generating a changelog
* Modify the code to run post-changelog scripts after updating the changelog
* Update the function to apply the changelog and handle errors appropriately
* Implement post-bump script execution and error handling

This commit includes modifications to the internal/cmd/bump.go file, where changes have been made to run pre-changelog scripts before generating a changelog, implement post-changelog script execution and error handling after updating the changelog, and apply the changelog with proper error handling. Additionally, this commit includes running post-bump scripts and error handling for any issues that may arise during their execution.
- Implemented hooks for pre-bump, post-bump, pre-changelog and post-changelog processes in root directory
@sergiotejon sergiotejon self-assigned this Nov 7, 2024
@sergiotejon sergiotejon linked an issue Nov 7, 2024 that may be closed by this pull request
Copy link
Contributor

@Muriano Muriano left a comment

Choose a reason for hiding this comment

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

Guapízimo

@sergiotejon sergiotejon merged commit 67abbf9 into main Nov 8, 2024
@sergiotejon sergiotejon deleted the feat/hooks branch November 8, 2024 09:52
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.

Gommitizen Hooks (similiar to git hooks)
2 participants