FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Improve status output by SandrineP · Pull Request #55 · QuantStack/git2cpp · GitHub

  1. QuantStack
  2. git2cpp

Repository navigation

Improve status output #55

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.

Sign up for GitHub

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

Merged
ianthomas23 merged 3 commits into from
Nov 10, 2025
Conversation 5 Commits 3 Checks 1 Files changed
Merged

Improve status output #55

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
828adc2
improve status output
SandrineP Nov 3, 2025
126d4d4
address review comments
SandrineP Nov 10, 2025
5634d20
address review comments
SandrineP Nov 10, 2025
File filter

Filter by extension

Filter by extension .cpp  (1) All 1 file type selected
Viewed files
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Unified
Split
Hide whitespace
Diff view
Unified
Split
Hide whitespace
61 changes: 44 additions & 17 deletions src/subcommand/status_subcommand.cpp
Show comments
View file
Open in desktop
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Original file line number Diff line number Diff line change
Expand Up @@ -26,16 +26,18 @@ status_subcommand::status_subcommand(const libgit2_object&, CLI::App& app)
sub->callback([this]() { this->run(); });
};

const std::string untracked_header = "Untracked files:\n";
// "Untracked files:\n (use \"git add <file>...\" to include in what will be committed)";
const std::string untracked_header = "Untracked files:\n (use \"git add <file>...\" to include in what will be committed)\n";
const std::string tobecommited_header = "Changes to be committed:\n";
// "Changes to be committed:\n (use \"git reset HEAD <file>...\" to unstage)";
const std::string ignored_header = "Ignored files:\n";
// "Ignored files:\n (use \"git add -f <file>...\" to include in what will be committed)"
const std::string notstagged_header = "Changes not staged for commit:\n";
// "Changes not staged for commit:\n (use \"git add%s <file>...\" to update what will be committed)\n (use \"git checkout -- <file>...\" to discard changes in working directory)"
const std::string nothingtocommit_message = "No changes added to commit";
// "No changes added to commit (use \"git add\" and/or \"git commit -a\")"
// (use \"git restore --staged <file>...\" to unstage)\n
// (use \"git reset HEAD <file>...\" to unstage)\n";
// const std::string ignored_header = "Ignored files:\n (use \"git add -f <file>...\" to include in what will be committed)\n";
const std::string notstagged_header = "Changes not staged for commit:\n (use \"git add <file>...\" to update what will be committed)\n";
// (use \"git restore <file>...\" to discard changes in working directory)\n
// (use \"git checkout -- <file>...\" to discard changes in working directory)\n"
const std::string nothingtocommit_msg = "No changes added to commit (use \"git add\" and/or \"git commit -a\")";
const std::string uptodate_msg = "Nothing to commit, working tree clean.";
const std::string nothingtocommit_untrackedfiles_msg = "Nothing added to commit but untracked files present (use \"git add\" to track)";
// no changes added to commit (use "git add" and/or "git commit -a")

struct status_messages
{
Expand All @@ -51,7 +53,7 @@ const std::map<git_status_t, status_messages> status_msg_map = //TODO : check
{ GIT_STATUS_INDEX_DELETED, {"D ", "\tdeleted:"} },
{ GIT_STATUS_INDEX_RENAMED, {"R ", "\trenamed:"} },
{ GIT_STATUS_INDEX_TYPECHANGE, {"T ", "\ttypechange:"} },
{ GIT_STATUS_WT_NEW, {"?? ", ""} },
{ GIT_STATUS_WT_NEW, {"?? ", " "} },
{ GIT_STATUS_WT_MODIFIED, {" M " , "\tmodified:"} },
{ GIT_STATUS_WT_DELETED, {" D ", "\tdeleted:"} },
{ GIT_STATUS_WT_TYPECHANGE, {" T ", "\ttypechange:"} },
Expand Down Expand Up @@ -79,7 +81,14 @@ std::string get_print_status(git_status_t status, output_format of)
std::string entry_status;
if ((of == output_format::DEFAULT) || (of == output_format::LONG))
{
entry_status = status_msg_map.at(status).long_mod + " ";
if (status == GIT_STATUS_WT_NEW)
{
entry_status = status_msg_map.at(status).long_mod + "\t";
}
else
{
entry_status = status_msg_map.at(status).long_mod + " ";
}
}
else if (of == output_format::SHORT)
{
Expand All @@ -88,14 +97,14 @@ std::string get_print_status(git_status_t status, output_format of)
return entry_status;
}

void update_tracked_dir_set(const char* old_path, const char* new_path, std::set<std::string>* tracked_dir_set = nullptr)
void update_tracked_dir_set(const char* path, std::set<std::string>* tracked_dir_set = nullptr)
{
if (tracked_dir_set)
{
const size_t first_slash_idx = std::string_view(old_path).find('/');
const size_t first_slash_idx = std::string_view(path).find('/');
if (std::string::npos != first_slash_idx)
{
auto directory = std::string_view(old_path).substr(0, first_slash_idx);
auto directory = std::string_view(path).substr(0, first_slash_idx);
tracked_dir_set->insert(std::string(directory));
}
}
Expand Down Expand Up @@ -131,7 +140,7 @@ std::vector<print_entry> get_entries_to_print(git_status_t status, status_list_w
const char* old_path = diff_delta->old_file.path;
const char* new_path = diff_delta->new_file.path;

update_tracked_dir_set(old_path, new_path, tracked_dir_set);
update_tracked_dir_set(old_path, tracked_dir_set);

print_entry e = { get_print_status(status, of), get_print_item(old_path, new_path)};

Expand Down Expand Up @@ -164,7 +173,8 @@ void print_not_tracked(const std::vector<print_entry>& entries_to_print, const s
const size_t first_slash_idx = e.item.find('/');
if (std::string::npos != first_slash_idx)
{
auto directory = "\t" + e.item.substr(0, first_slash_idx) + "/";
auto directory = e.item.substr(0, first_slash_idx);
auto directory_print = e.item.substr(0, first_slash_idx) + "/";
if (tracked_dir_set.contains(directory))
{
not_tracked_entries_to_print.push_back(e);
Expand All @@ -175,7 +185,7 @@ void print_not_tracked(const std::vector<print_entry>& entries_to_print, const s
{}
else
{
not_tracked_entries_to_print.push_back({e.status, directory});
not_tracked_entries_to_print.push_back({e.status, directory_print});
untracked_dir_set.insert(std::string(directory));
}
}
Expand Down Expand Up @@ -290,4 +300,21 @@ void status_subcommand::run()
// std::cout << std::endl;
// }
// }

if (!sl.has_tobecommited_header() && (sl.has_notstagged_header() || sl.has_untracked_header()))
{
if (sl.has_untracked_header())
{
std::cout << nothingtocommit_untrackedfiles_msg << std::endl;
}
else
{
std::cout << nothingtocommit_msg << std::endl;
}
}

if (!sl.has_notstagged_header() && !sl.has_untracked_header())
{
std::cout << uptodate_msg << std::endl;
}
}
Back | FazBrowse Home | New Git URL