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

Remove the AlwaysShowNews option from the config file #284

Closed
Antiz96 opened this issue Dec 2, 2024 · 0 comments · Fixed by #288
Closed

Remove the AlwaysShowNews option from the config file #284

Antiz96 opened this issue Dec 2, 2024 · 0 comments · Fixed by #288
Assignees
Labels
good first issue Good for newcomers question/feedback General question/feedback
Milestone

Comments

@Antiz96
Copy link
Owner

Antiz96 commented Dec 2, 2024

This option was introduced to keep compatibility with the historic & initial Arch-Update's behavior of always showing news on upgrade (regardless of whether there's a new one since the last run or not). I think it's fair to say that this was a poor design that just created undesired annoyances to people.

As such, I'm not sure the AlwaysShowNews option is really desired / used by anyone and therefore could be deprecated & removed altogether. The current default behavior of showing news only if a new one has been published since the last run feels like a sane one.

If anything, news can be displayed at any time by running arch-update -n / arch-update --news.

@Antiz96 Antiz96 added good first issue Good for newcomers question/feedback General question/feedback labels Dec 2, 2024
@Antiz96 Antiz96 self-assigned this Dec 2, 2024
@Antiz96 Antiz96 added this to the v3.4.2 milestone Dec 4, 2024
Antiz96 added a commit that referenced this issue Dec 6, 2024
The AlwaysShowNews was only introduced to keep compatibility with the historic & initial Arch-Update's behavior of always showing news on upgrade (regardless of whether there's a new one since the last run or not). I think it's fair to say that this was a poor design that just created undesired annoyances to people.

As such, I'm not sure the AlwaysShowNews option is really desired / used by anyone and therefore could be deprecated & removed altogether.

Closes #284

---------

Co-authored-by: Robin Candau <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers question/feedback General question/feedback
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant