Skip to content
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

Jump to bottom

great_expectations checkpoint list truncating name #1518

Closed
anthonyburdi opened this issue Jun 1, 2020 · 3 comments
Closed

great_expectations checkpoint list truncating name #1518

anthonyburdi opened this issue Jun 1, 2020 · 3 comments
Labels
bug Bugs bugs bugs!

Comments

@anthonyburdi
Copy link
Member

Describe the bug
When running great_expectations checkpoint list the checkpoint name is truncated to either 14 characters or the last character is removed (I am not sure which).

To Reproduce
Steps to reproduce the behavior:

  1. Create a checkpoint with at least 15 characters
  2. Run great_expectations checkpoint list

Expected behavior
The list should be accurate based on the checkpoint names

Environment (please complete the following information):

  • OS: Ubuntu 18.04.4 LTS
  • GE Version: 0.11.0
@eugmandel eugmandel added the bug Bugs bugs bugs! label Jun 1, 2020
@eugmandel
Copy link
Contributor

Confirmed.

@Aylr
Copy link
Contributor

Aylr commented Jun 3, 2020

great_expectations checkpoint list appears to strip off all ls so if you have a checkpoint named lllll it will show up as -

eugmandel added a commit that referenced this issue Jun 3, 2020
@eugmandel
Fixed issue #1518
73a26e0
eugmandel added a commit that referenced this issue Jun 3, 2020
@eugmandel
Fixed issue #1518
296bcf7
@eugmandel
Copy link
Contributor

@Aylr , yes, "rstrip" was used to remove the yml file extension. Fixing.

@eugmandel eugmandel mentioned this issue Jun 3, 2020
Merged
eugmandel added a commit that referenced this issue Jun 5, 2020
@eugmandel @jcampbell
[BUGFIX] Fixed issue where CLI checkpoint list truncated names (#1518) (
3c0a3de 
#1541)

* Fixed issue #1518

* Fixed issue #1518

* Update docs/reference/changelog.rst

Co-authored-by: James Campbell <james.p.campbell@gmail.com>

* linter

Co-authored-by: James Campbell <james.p.campbell@gmail.com>
@Aylr Aylr closed this as completed Jul 2, 2020
alexsherstinsky pushed a commit to alexsherstinsky/great_expectations that referenced this issue Feb 19, 2021
@eugmandel @jcampbell
[BUGFIX] Fixed issue where CLI checkpoint list truncated names (great…
fabc451 
…-expectations#1518) (great-expectations#1541)

* Fixed issue great-expectations#1518

* Fixed issue great-expectations#1518

* Update docs/reference/changelog.rst

Co-authored-by: James Campbell <james.p.campbell@gmail.com>

* linter

Co-authored-by: James Campbell <james.p.campbell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Bugs bugs bugs!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@eugmandel @Aylr @anthonyburdi