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

Make CommitList display "Working tree clean" #708

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

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
Copy link
Contributor Author

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

@Murmele
Copy link
Owner

Murmele commented May 22, 2024

Now this can be rebased as well :)

@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...

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