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

Gtk4 skeleton #2549

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Gtk4 skeleton #2549

wants to merge 16 commits into from

Conversation

jeremypw
Copy link
Collaborator

@jeremypw jeremypw commented Mar 13, 2025

The intention of this is to provide an intermediate branch which compiles (but does not necessarily run atm) with Gtk4 to which small branches may be pushed porting different aspects of the code. This allows for a series of manageable reviews and approvals.

The non-compiling code has the for the most part been commented out except where a very simple fix is possible or it was not possible to fix by just commenting code out.

Branches pushed to this branch should incrementally replace commented out code with working Gtk4 code with as small a diff as practical.

As of 745e1a7, this compiles and runs with no terminal messages - but only shows an empty window.

@danirabbit Is there a way of having PRs that are to be merged into this branch require at least 1 approving review?

--PRs be merged--

@jeremypw jeremypw added the Status: Blocked Cannot proceed until another issue is fixed label Mar 13, 2025
@danirabbit
Copy link
Member

Is there a way of having PRs that are to be merged into this branch require at least 1 approving review
@jeremypw yes, but I'm not sure why you want to do that?

@jeremypw
Copy link
Collaborator Author

Is there a way of having PRs that are to be merged into this branch require at least 1 approving review
@jeremypw yes, but I'm not sure why you want to do that?

Because I want this PR not to require detailed review as a whole because the diff will be large. If the merged PRs have already been independently reviewed at least for coding rationality then this PR will only need reviewing for overall functionality and appearance.

# Conflicts fixed:
#	filechooser-portal/Main.vala
#	libcore/AbstractSlot.vala
#	libcore/FileUtils.vala
#	plugins/pantheon-files-trash/plugin.vala
#	src/Dialogs/ChooseAppDialog.vala
#	src/View/DirectoryNotFound.vala
#	src/View/Window.vala
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Blocked Cannot proceed until another issue is fixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants