Skip to content

Commit

Permalink
Merge pull request #77 from ihuicatl/tabulate
Browse files Browse the repository at this point in the history
Update cpp-tabulate dependency to v1.5
  • Loading branch information
JohanMabille authored Nov 14, 2023
2 parents 67eb6ed + 15857fa commit 1ac9e6d
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ The xeus team organizes public video meetings. The schedule for future meetings
To install the xeus-sql dependencies:

```bash
mamba install nlohmann_json xtl cppzmq xeus cpp-tabulate=1.3 xvega xvega-bindings xproperty jupyterlab soci-core compilers cmake -c conda-forge
mamba install nlohmann_json xtl cppzmq xeus cpp-tabulate>=1.5 xvega xvega-bindings xproperty jupyterlab soci-core compilers cmake -c conda-forge
```

#### Known issues
Expand Down
4 changes: 2 additions & 2 deletions docs/source/installation.rst
Original file line number Diff line number Diff line change
Expand Up @@ -61,11 +61,11 @@ You can install ``xeus-sql`` from source with ``cmake``. This requires that you

.. code::
conda install cmake nlohmann_json xtl cppzmq xeus mysql sqlite postgresql cpp-tabulate=1.3 xvega xvega-bindings xproperty jupyterlab compilers -c conda-forge
conda install cmake nlohmann_json xtl cppzmq xeus mysql sqlite postgresql cpp-tabulate>=1.5 xvega xvega-bindings xproperty jupyterlab compilers -c conda-forge
.. code::
mamba install cmake nlohmann_json xtl cppzmq xeus mysql sqlite postgresql cpp-tabulate=1.3 xvega xvega-bindings xproperty jupyterlab compilers -c conda-forge
mamba install cmake nlohmann_json xtl cppzmq xeus mysql sqlite postgresql cpp-tabulate>=1.5 xvega xvega-bindings xproperty jupyterlab compilers -c conda-forge
.. code::
Expand Down
2 changes: 1 addition & 1 deletion environment-dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ dependencies:
- cxx-compiler
- ninja
# host dependencies
- cpp-tabulate=1.3
- cpp-tabulate>=1.5
- cppzmq
- nlohmann_json
- soci-core
Expand Down
4 changes: 2 additions & 2 deletions src/xeus_sql_interpreter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ namespace xeus_sql
for (const soci::row& r : rows)
{
if (row_count == 0) {
std::vector<variant<std::string, const char*, tabulate::Table>> col_names;
tabulate::Table::Row_t col_names;
html_table << "<table>\n<tr>\n";
for (std::size_t i = 0; i != r.size(); ++i) {
std::string name = r.get_properties(i).get_name();
Expand All @@ -108,7 +108,7 @@ namespace xeus_sql
/* Iterates through cols' rows and builds different kinds of
outputs
*/
std::vector<variant<std::string, const char*, tabulate::Table>> row;
tabulate::Table::Row_t row;
html_table << "<tr>\n";
for(std::size_t i = 0; i != r.size(); ++i)
{
Expand Down

0 comments on commit 1ac9e6d

Please sign in to comment.