Skip to content

Add commit and reset subcommand #29

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 17 commits into from
Jul 31, 2025
Merged

Add commit and reset subcommand #29

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
b977235
add commit subcommand
SandrineP Jul 29, 2025
4dbb90d
add reset subcommand
SandrineP Jul 29, 2025
da951b1
write index
SandrineP Jul 30, 2025
eef5385
fix
SandrineP Jul 30, 2025
0c7c928
small fix
SandrineP Jul 30, 2025
7ee91ab
test commit
SandrineP Jul 30, 2025
8082673
update test
SandrineP Jul 30, 2025
516dba5
remove test file
SandrineP Jul 30, 2025
1076d42
fix test
SandrineP Jul 30, 2025
75d16ee
add branch to test
SandrineP Jul 30, 2025
fee3016
edit branch name
SandrineP Jul 30, 2025
8b9eaf4
remove mook test file
SandrineP Jul 30, 2025
516e726
test commit
SandrineP Jul 30, 2025
2ba0ac9
edit file path
SandrineP Jul 30, 2025
954587d
address review comments
SandrineP Jul 31, 2025
d6c35c3
edit option
SandrineP Jul 31, 2025
67f57dc
remove extra space
SandrineP Jul 31, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
edit option
  • Loading branch information
@SandrineP
SandrineP committed Jul 31, 2025
commit d6c35c3667b7e1cbd01da52af4b339f5ba4a8ad7
17 changes: 7 additions & 10 deletions src/subcommand/commit_subcommand.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,7 @@ commit_subcommand::commit_subcommand(const libgit2_object&, CLI::App& app)
{
auto *sub = app.add_subcommand("commit", "Record changes to the repository");

// TODO: merge the following option and flag into: sub->add_option("-m,--message", m_commit_message, "Commit message")
// when we have a solution to ask for a message when not using the -m flag.
sub->add_option("commit_message", m_commit_message, "Commit message");

sub->add_flag("-m,--message", m_commit_message_flag, "");
sub->add_option("-m,--message", m_commit_message, "Commit message");

sub->callback([this]() { this->run(); });
};
Expand All @@ -27,13 +23,14 @@ void commit_subcommand::run()
auto repo = repository_wrapper::init(directory, bare);
auto author_committer_signatures = signature_wrapper::get_default_signature_from_env(repo);

if (!m_commit_message_flag)
{
throw std::runtime_error("Please provide a message using the -m flag.");
}
if (m_commit_message.empty())
{
throw std::runtime_error("Please provide a commit message.");
std::cout << "Please enter a commit message: " << std::endl;
std::getline(std::cin, m_commit_message);
if (m_commit_message.empty())
{
throw std::runtime_error("Aborting, no commit message specified.");
}
}

repo.create_commit(author_committer_signatures, m_commit_message);
Expand Down
1 change: 0 additions & 1 deletion src/subcommand/commit_subcommand.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,5 @@ class commit_subcommand
void run();

private:
bool m_commit_message_flag = true; // TODO: change to false when a message can be provided if the "-m" flag is not provided
std::string m_commit_message;
};
Loading