https://bugzilla.redhat.com/show_bug.cgi?id=2428626
--- Comment #5 from Carl George 🤠<[email protected]> --- > This should be using the new dependency generator stuff I intentionally only want to set a lower bound. This allows for users to easily test the extension with a newer GNOME version by setting the dconf key /org/gnome/shell/disable-extension-version-validation to true. And if they don't set that, they can still disable the extension without uninstalling it and proceed with system upgrades. When a future update adds compatibility they can observe in the Extensions app that the warning icon goes away and they can enable it again. Beyond just using the extension, this capability is highly beneficial for collaboration with upstream. It makes it easy to report to upstream that the extension works on new versions of GNOME (so they just need to update metadata.json), or that it doesn't work with the corresponding error message. Both of those aspects, combined with the quick six month cadence of new GNOME versions, makes the generator untenable to me. Since it's not required by the guidelines, I'll pass on this. > This should have a trailing slash This is also not required by the guidelines, and is just a personal preference, so I'll pass on this one too. Are there any review-blocking items? -- You are receiving this mail because: You are always notified about changes to this product and component You are on the CC list for the bug. https://bugzilla.redhat.com/show_bug.cgi?id=2428626 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202428626%23c5 -- _______________________________________________ package-review mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/[email protected] Do not reply to spam, report it: https://forge.fedoraproject.org/infra/tickets/issues/new
