Skip to content
This repository has been archived by the owner on May 11, 2023. It is now read-only.

Add changelog #260

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add changelog #260

wants to merge 2 commits into from

Conversation

erak
Copy link
Contributor

@erak erak commented Sep 26, 2022

Closes #249.

@erak erak requested a review from cloudhead as a code owner September 26, 2022 22:49
- `rad-auth`: Remove passphrase option, read passphrase from environment variable `RAD_PASSPHRASE` or from stdin

## [0.6.1] - 2022-08-11

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@adaszko I'd need your help to sort out this section. Seems like the release artifacts are somehow out of sync. The release dates for 0.6.0 and 0.6.1 where taken from Github's release page. v0.6.0...v0.6.1 only contains the release commit. Whereas v0.6.1...master contains the actual commits that happened between the release dates. Could you have a look?

Copy link
Contributor

Choose a reason for hiding this comment

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

There was a mess-up: 0.6.0 was not pointing to a commit in the actual history, so we had to create a new 0.6.1 tag pointing to the commit. 0.6.0 and 0.6.1 are the same code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I think I just found a hint in the release notes... So, I'd then keep the 0.6.1 section empty and move all the items to 0.6.0.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes for 0.6.1 just say No changes.

Signed-off-by: Erik Kundt <[email protected]>
@erak erak force-pushed the erak/changelog branch 2 times, most recently from bed3096 to e2ce4ed Compare September 30, 2022 13:33
CHANGELOG.md Outdated
## [Unreleased]
### Added
- Add Dockerfile for `radicle-cli`
- Re-organize binaries into own crate and move all sub-commands to `rad` binary
Copy link
Contributor

Choose a reason for hiding this comment

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

Who is the Changelog directed at? I ask because this appears dev facing

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ya, agree. So I tried to find a middle ground between a changelog for devs / technical writers and end users. But I guess your comment and the reactions to it smell like necessary simplification.

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.

Add changelog
3 participants