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: move gnome-extension to stable #4799

Merged
merged 3 commits into from
Jun 17, 2024
Merged

chore: move gnome-extension to stable #4799

merged 3 commits into from
Jun 17, 2024

Conversation

jssotomdz
Copy link
Contributor

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox run -m lint?
  • Have you successfully run tox run -e test-py310? (supported versions: py39, py310, py311, py312)

This PR will have to wait for the support for streamlined graphics as stated by #4736
We want to have it ready for that so we can start migrating desktop snaps to core24 when that is done

@jssotomdz
Copy link
Contributor Author

@kenvandine

Copy link
Contributor

@kenvandine kenvandine left a comment

Choose a reason for hiding this comment

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

Looks good to me

@nteodosio
Copy link
Contributor

I'm very interested in this as Chromium Dev now requires a new enough version of Pipewire not available in core22 and the alternative of building Pipewire from source is not so nice.

Copy link
Collaborator

@mr-cal mr-cal left a comment

Choose a reason for hiding this comment

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

@jssotomdz, can you drop the changes to snapcraft/services/provider.py and tests/unit/services/test_provider.py?

That code is not specific to the gnome extension, it's generic code for any experimental extension.

@jssotomdz
Copy link
Contributor Author

can you drop the changes to snapcraft/services/provider.py and tests/unit/services/test_provider.py?

done!

@mr-cal mr-cal requested a review from a team June 13, 2024 15:07
@mr-cal mr-cal merged commit 14dbf54 into canonical:main Jun 17, 2024
10 checks passed
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.

5 participants