feat: prettier table

src/command/graph/check.rs

@@ -1,4 +1,4 @@
-use prettytable::{cell, row, Table};
+use prettytable::{cell, row};
 use serde::Serialize;
 use structopt::StructOpt;
 
@@ -12,6 +12,7 @@ use crate::utils::parsers::{
     parse_graph_ref, parse_query_count_threshold, parse_query_percentage_threshold,
     parse_schema_source, parse_validation_period, GraphRef, SchemaSource, ValidationPeriod,
 };
+use crate::utils::table;
 use crate::Result;
 
 #[derive(Debug, Serialize, StructOpt)]
@@ -111,8 +112,8 @@ fn print_changes(
     let mut num_failures = 0;
 
     if !checks.is_empty() {
-        let mut table = Table::new();
-        table.add_row(row!["Change", "Code", "Description"]);
+        let mut table = table::get_table();
+        table.add_row(row![bc => "Change", "Code", "Description"]);
         for check in checks {
             let change = match check.severity {
                 check::check_schema_query::ChangeSeverity::NOTICE => "PASS",

src/command/output.rs

@@ -1,8 +1,10 @@
 use std::fmt::Debug;
 
-use prettytable::{cell, row, Table};
+use prettytable::{cell, row};
 use rover_client::query::subgraph::list::ListDetails;
 
+use crate::utils::table;
+
 /// RoverStdout defines all of the different types of data that are printed
 /// to `stdout`. Every one of Rover's commands should return `anyhow::Result<RoverStdout>`
 /// If the command needs to output some type of data, it should be structured
@@ -32,8 +34,8 @@ impl RoverStdout {
                 println!("{}", &hash);
             }
             RoverStdout::SubgraphList(details) => {
-                let mut table = Table::new();
-                table.add_row(row!["Name", "Routing Url", "Last Updated"]);
+                let mut table = table::get_table();
+                table.add_row(row![bc => "Name", "Routing Url", "Last Updated"]);
 
                 for subgraph in &details.subgraphs {
                     // if the url is None or empty (""), then set it to "N/A"

src/command/subgraph/check.rs

@@ -1,4 +1,4 @@
-use prettytable::{cell, row, Table};
+use prettytable::{cell, row};
 use serde::Serialize;
 use structopt::StructOpt;
 
@@ -13,6 +13,7 @@ use crate::utils::parsers::{
     parse_graph_ref, parse_query_count_threshold, parse_query_percentage_threshold,
     parse_schema_source, parse_validation_period, GraphRef, SchemaSource, ValidationPeriod,
 };
+use crate::utils::table;
 
 #[derive(Debug, Serialize, StructOpt)]
 pub struct Check {
@@ -119,8 +120,8 @@ fn handle_checks(check_result: check::CheckResult) -> Result<RoverStdout> {
     let mut num_failures = 0;
 
     if !check_result.changes.is_empty() {
-        let mut table = Table::new();
-        table.add_row(row!["Change", "Code", "Description"]);
+        let mut table = table::get_table();
+        table.add_row(row![bc => "Change", "Code", "Description"]);
         for check in check_result.changes {
             let change = match check.severity {
                 check::check_partial_schema_query::ChangeSeverity::NOTICE => "PASS",

src/utils/mod.rs

@@ -5,4 +5,5 @@ pub mod loaders;
 pub mod parsers;
 pub mod pkg;
 pub mod stringify;
+pub mod table;
 pub mod telemetry;

src/utils/table.rs

@@ -0,0 +1,25 @@
+use prettytable::{
+    format::{FormatBuilder, LinePosition, LineSeparator},
+    Table,
+};
+
+pub fn get_table() -> Table {
+    let mut table = Table::new();
+    table.set_format(
+        FormatBuilder::new()
+            .column_separator('│')
+            .borders('│')
+            .separators(&[LinePosition::Top], LineSeparator::new('─', '┬', '┌', '┐'))
+            .separators(
+                &[LinePosition::Intern],
+                LineSeparator::new('─', '┼', '├', '┤'),
+            )
+            .separators(
+                &[LinePosition::Bottom],
+                LineSeparator::new('─', '┴', '└', '┘'),
+            )
+            .padding(1, 1)
+            .build(),
+    );
+    table
+}