Skip to content
This repository was archived by the owner on Oct 28, 2021. It is now read-only.

Fix changelog bug and add entry for PR #5538#5541

Merged
halfalicious merged 2 commits intomasterfrom
changelog-updates
Apr 3, 2019
Merged

Fix changelog bug and add entry for PR #5538#5541
halfalicious merged 2 commits intomasterfrom
changelog-updates

Conversation

@halfalicious
Copy link
Copy Markdown
Contributor

PR 5538 removed the --private option from aleth

@halfalicious halfalicious requested review from chfast and gumb0 and removed request for chfast April 2, 2019 03:19
Copy link
Copy Markdown
Member

@chfast chfast left a comment

Choose a reason for hiding this comment

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

Also keep the list grouped by "kind": Added, Changed, Removed, Fixed, ...

## [1.6.0] - Unreleased

- Added: [#5484](https://github.com/ethereum/aleth/pull/5485) aleth-bootnode now by default connects to official Ethereum bootnodes. This can be disabled with `--no-bootstrap` flag.
- Added: [#5485](https://github.com/ethereum/aleth/pull/5485) aleth-bootnode now by default connects to official Ethereum bootnodes. This can be disabled with `--no-bootstrap` flag.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I believe this also should be Changed: as it is not a new feature.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why not, this is new code and new flag added

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can be. It rather feels to me we are changing only how features are enabled / disabled. But I don't care much.

PR 5538 removed the --private option from aleth
Grouped entries by kind and changed "added" to "removed" for PR which
removed a command-line argument.
@halfalicious
Copy link
Copy Markdown
Contributor Author

Rebased to address merge conflict

@halfalicious
Copy link
Copy Markdown
Contributor Author

Also keep the list grouped by "kind": Added, Changed, Removed, Fixed, ...

Did that in the new version (grouped changes in that order: Added, Changed, Removed, Fixed)

@halfalicious halfalicious requested a review from chfast April 2, 2019 15:13
@halfalicious halfalicious merged commit ad7204c into master Apr 3, 2019
@halfalicious halfalicious deleted the changelog-updates branch April 3, 2019 02:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants