Skip to content

Make CommitList display "Working tree clean"#708

Draft
jensenr30 wants to merge 1 commit intoMurmele:masterfrom
jensenr30:working-tree-clean
Draft

Make CommitList display "Working tree clean"#708
jensenr30 wants to merge 1 commit intoMurmele:masterfrom
jensenr30:working-tree-clean

Conversation

@jensenr30
Copy link
Contributor

@jensenr30 jensenr30 commented Mar 5, 2024

When there are no changes in the working directory, the top placeholder commit says "Working tree clean".

I added a function bool is_working_tree_clean(void) const in class CommitModel : public QAbstractListModel. Is there a better place to put this function? I'm a C++ beginner. I was trying to figure out how to access the is_working_tree_clean() function in other places in CommitList.cpp but couldn't figure out a clean way to do it...

image

Resolves #704

@jensenr30 jensenr30 marked this pull request as draft March 9, 2024 05:04
@Murmele
Copy link
Owner

Murmele commented Mar 15, 2024

Thanks for the changes. Is this ready?
I think the is_working_tree_clean function is fine there

@jensenr30
Copy link
Contributor Author

@Murmele
I think this is good to merge in, but it might be nice to merge #714 first, because that fixes a bug with this element. But it's your call of course.

@jensenr30 jensenr30 force-pushed the working-tree-clean branch from 0d30464 to 13544c2 Compare April 4, 2024 04:49
@jensenr30
Copy link
Contributor Author

@Murmele
I will leave this as a Draft until #714 gets merged in because this depends on #714.

@jensenr30 jensenr30 force-pushed the working-tree-clean branch from 13544c2 to fd3bd4f Compare April 20, 2024 21:10
@Murmele
Copy link
Owner

Murmele commented May 22, 2024

Now this can be rebased as well :)

@jensenr30 jensenr30 force-pushed the working-tree-clean branch from fd3bd4f to 0c549c7 Compare May 22, 2024 08:17
@jensenr30 jensenr30 marked this pull request as ready for review May 22, 2024 08:18
@jensenr30
Copy link
Contributor Author

Hmmm... Looks like I broke some tests... I'll look into this...

@Murmele Murmele marked this pull request as draft September 12, 2025 09:47
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.

Enhance Uncommitted Changes to show git status

2 participants