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

chore(build-dx): consolidate os-release names #71

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

ledif
Copy link
Collaborator

@ledif ledif commented Jan 3, 2025

This PR contains two separate but related changes:

Use same PRETTY_NAME for Aurora and Aurora-dx

Mostly pulling from Bluefin's recent change ublue-os/bluefin#2090 to make it so both Aurora and Aurora-dx count as a single entry in Homebrew's stats.

This should bump Aurora up a few spots on https://formulae.brew.sh/analytics/os-version/30d.

Change name in KDE's Info Center

The kcm-about-distrorc file controls what is displayed in KDE's Info Center app. I noticed that Aurora's Info Center is using the Variant field in a way that is both redundant (i.e., it is just the Name field repeated) and different from how other KDE-based projects use that field, including Bazzite.

Current

Screenshot_20250103_012617

Proposed

Screenshot_20250103_022127

I feel that having the Variant field be empty for the Aurora image and "Developer Experience" for the Aurora-dx image is more in line with the spirit of the field. I am attaching examples from other distros to provide a feel for how others do it.

Other Distros

Screenshot_20250103_020847
Screenshot_20250103_012224
Screenshot_20250103_013005
Screenshot_20250103_012456

@ledif ledif requested a review from castrojo as a code owner January 3, 2025 08:45
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 3, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 3, 2025
@NiHaiden NiHaiden added this pull request to the merge queue Jan 3, 2025
Merged via the queue into ublue-os:main with commit 5ba9b90 Jan 3, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants