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

docs: add missing password manager removal to changelog #1572

Merged
merged 1 commit into from
Dec 30, 2024

Conversation

ThomasFrans
Copy link
Contributor

As this is a breaking change, it should definitely be represented in the changelog, which is the primary source of information about any user-facing changes.

Describe your changes

Add the removal of the password manager feature to the "Removed" section of the changelog.
Also update the wording for another item in the "Fixed" section to say exactly what was fixed.

Issue ticket number and link

/

Checklist before requesting a review

  • Documentation was updated (i.e. due to changes in keybindings, commands, etc.)
  • Changelog was updated with relevant user-facing changes (eg. not dependency updates,
    not performance improvements, etc.)

(I'll also create a separate PR to remove some remaining documentation about the password manager feature.)

As this is a breaking change, it should definitely be represented in the
changelog, which is the primary source of information about any
user-facing changes.
@hrkfdn hrkfdn merged commit fa7b698 into hrkfdn:main Dec 30, 2024
5 checks passed
@hrkfdn
Copy link
Owner

hrkfdn commented Dec 30, 2024

Makes sense, thank you!

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.

2 participants