aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFrancois Kritzinger <francois@codesynthesis.com>2024-10-31 15:17:31 +0200
committerFrancois Kritzinger <francois@codesynthesis.com>2024-11-04 11:10:53 +0200
commit929ee20752c4430af3002814fb12329b157574bb (patch)
tree54f201f7949a2339f5c4ae5d6d515f184873bc40
parentfa35889ff0bcf47bcf1955eb784cb43f7890e5a1 (diff)
Remove unused member gh_pull_request::state
-rw-r--r--mod/mod-ci-github-gh.cxx5
-rw-r--r--mod/mod-ci-github-gh.hxx2
2 files changed, 1 insertions, 6 deletions
diff --git a/mod/mod-ci-github-gh.cxx b/mod/mod-ci-github-gh.cxx
index 8af3403..6e178e2 100644
--- a/mod/mod-ci-github-gh.cxx
+++ b/mod/mod-ci-github-gh.cxx
@@ -186,7 +186,7 @@ namespace brep
{
p.next_expect (event::begin_object);
- bool ni (false), nu (false), st (false), bs (false), hd (false);
+ bool ni (false), nu (false), bs (false), hd (false);
// Skip unknown/uninteresting members.
//
@@ -199,7 +199,6 @@ namespace brep
if (c (ni, "node_id")) node_id = p.next_expect_string ();
else if (c (nu, "number")) number = p.next_expect_number<unsigned int> ();
- else if (c (st, "state")) state = p.next_expect_string ();
else if (c (bs, "base"))
{
p.next_expect (event::begin_object);
@@ -265,7 +264,6 @@ namespace brep
if (!ni) missing_member (p, "gh_pull_request", "node_id");
if (!nu) missing_member (p, "gh_pull_request", "number");
- if (!st) missing_member (p, "gh_pull_request", "state");
if (!bs) missing_member (p, "gh_pull_request", "base");
if (!hd) missing_member (p, "gh_pull_request", "head");
}
@@ -275,7 +273,6 @@ namespace brep
{
os << "node_id: " << pr.node_id
<< ", number: " << pr.number
- << ", state: " << pr.state
<< ", base: { "
<< "path: " << pr.base_path
<< ", ref: " << pr.base_ref
diff --git a/mod/mod-ci-github-gh.hxx b/mod/mod-ci-github-gh.hxx
index 16d4ff5..f79d01f 100644
--- a/mod/mod-ci-github-gh.hxx
+++ b/mod/mod-ci-github-gh.hxx
@@ -73,8 +73,6 @@ namespace brep
string node_id;
unsigned int number;
- string state; // "open" or "closed".
-
// @@ TMP The unused base/head members may be useful for trace output when
// we receive the pull_request webhook.