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

Refactor builder & changelog in to pkg #101

Merged
merged 4 commits into from
Oct 9, 2022
Merged

Refactor builder & changelog in to pkg #101

merged 4 commits into from
Oct 9, 2022

Conversation

chelnak
Copy link
Owner

@chelnak chelnak commented Oct 9, 2022

This commit moves builder in to it's own public package. Additionally the builder package has also been refactored.

The previous builder pattern has been dropped in favour of a standard options pattern.

Also methods have been broken down as much as possible with the hope that it will increase readability and help reduce duplication in the future.

Additionally, changelog has also been moved in to it's own public package. The code remains largely untouched.

Tests have also been updated, along with mocks, to reflect the new structure.

This commit moves builder in to it's own public package. Additionally
the builder package has also been refactored. The previous builder
pattern has been dropped in favour of a standard options pattern.

Also methods have been broken down as much as possible with the hope
that it will increase readability and help reduce duplication in the
future.

Tests have also been updated, along with mocks, to reflect the new
structure.
This commit moves changelog in to it's own public package. The code
remains largely untouched. However imports and tests have been updated,
along with mocks, to reflect the changes.
This commit rewires the writer package so that it can see the new
changelog package.
This commit rewires the new command so that it properly imports builder.
@chelnak chelnak added the enhancement New feature or request label Oct 9, 2022
@chelnak chelnak self-assigned this Oct 9, 2022
@chelnak chelnak merged commit 6e18898 into main Oct 9, 2022
@chelnak chelnak deleted the refactor_builder branch October 9, 2022 20:30
@chelnak chelnak linked an issue Oct 9, 2022 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the changelog builder package public
1 participant