Skip to content

fix(test): truncate output to fit line#309

Merged
felangel merged 2 commits intomainfrom
fix/test-output-truncated
Mar 11, 2022
Merged

fix(test): truncate output to fit line#309
felangel merged 2 commits intomainfrom
fix/test-output-truncated

Conversation

@felangel
Copy link
Contributor

@felangel felangel commented Mar 11, 2022

Description

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

@felangel felangel added the bug Something isn't working as expected label Mar 11, 2022
@felangel felangel requested a review from erickzanardo March 11, 2022 17:33
@felangel felangel self-assigned this Mar 11, 2022
@felangel felangel requested a review from jorgecoca as a code owner March 11, 2022 17:33
@felangel felangel force-pushed the fix/test-output-truncated branch from 6be6db4 to 4adfec7 Compare March 11, 2022 17:34
erickzanardo
erickzanardo previously approved these changes Mar 11, 2022
Copy link
Member

@erickzanardo erickzanardo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

} on StdoutException {
return 80;
}
}();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This gives me JS feelings haha

@felangel felangel merged commit 5487fcf into main Mar 11, 2022
@felangel felangel deleted the fix/test-output-truncated branch March 11, 2022 17:49
@felangel felangel mentioned this pull request Mar 17, 2022
7 tasks
ahsanf pushed a commit to Arkabyte-Teknologi/very_good_cli that referenced this pull request Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants