Skip to content

[elastic-package status] Consider disabling AutoMergeCell #360

@masci

Description

@masci

At https://github.com/elastic/elastic-package/blob/master/cmd/status.go#L143 we set table.SetAutoMergeCells(true) which should help reading the table but in my case was a source of confusion.

Take for example:

$ elastic-package status apache  -a
Package: apache
Package Versions:
+-------------+--------------------------------+--------------+--------+--------------------+
| ENVIRONMENT |            VERSION             |   RELEASE    | TITLE  |    DESCRIPTION     |
+-------------+--------------------------------+--------------+--------+--------------------+
| Snapshot    | 0.5.0 (0.1.3, 0.1.4, 0.2.6,    | experimental | Apache | Apache Integration |
|             | 0.3.0, 0.3.3, 0.3.4, 0.4.0)    |              |        |                    |
+-------------+                                +              +        +                    +
| Staging     |                                |              |        |                    |
|             |                                |              |        |                    |
+-------------+                                +              +        +                    +
| Production  |                                |              |        |                    |
|             |                                |              |        |                    |
+-------------+--------------------------------+--------------+--------+--------------------+

I thought the package wasn't deployed at all in environments other than Snapshot. If we disable the auto merge, the result would be:

$ elastic-package status apache -a
Package: apache
Package Versions:
+-------------+--------------------------------+--------------+--------+--------------------+
| ENVIRONMENT |            VERSION             |   RELEASE    | TITLE  |    DESCRIPTION     |
+-------------+--------------------------------+--------------+--------+--------------------+
| Snapshot    | 0.5.0 (0.1.3, 0.1.4, 0.2.6,    | experimental | Apache | Apache Integration |
|             | 0.3.0, 0.3.3, 0.3.4, 0.4.0)    |              |        |                    |
+-------------+--------------------------------+--------------+--------+--------------------+
| Staging     | 0.5.0 (0.1.3, 0.1.4, 0.2.6,    | experimental | Apache | Apache Integration |
|             | 0.3.0, 0.3.3, 0.3.4, 0.4.0)    |              |        |                    |
+-------------+--------------------------------+--------------+--------+--------------------+
| Production  | 0.5.0 (0.1.3, 0.1.4, 0.2.6,    | experimental | Apache | Apache Integration |
|             | 0.3.0, 0.3.3, 0.3.4, 0.4.0)    |              |        |                    |
+-------------+--------------------------------+--------------+--------+--------------------+

a bit more dense but IMO also more clear.

I can submit a patch if you'd consider the change.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions