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.

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

fix: Add new line while inspecting file contents #1477

Merged
merged 5 commits into from Oct 6, 2023
Merged

Conversation

victorcolombo
Copy link
Contributor

@victorcolombo victorcolombo commented Oct 5, 2023

Description:

Reader did not include break char (new lines), so the writer contained no new lines, giving the wrong output on both CLI and EM UI.

Is this change user facing?

YES

References (if applicable):

Closes #1417
Closes #1042

Copy link
Contributor

@tedim52 tedim52 left a comment

Choose a reason for hiding this comment

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

good catch

@tedim52 tedim52 enabled auto-merge (squash) October 6, 2023 12:24
@tedim52 tedim52 merged commit 545aa53 into main Oct 6, 2023
31 of 32 checks passed
@tedim52 tedim52 deleted the vcolombo/fix-nl branch October 6, 2023 12:59
h4ck3rk3y added a commit that referenced this pull request Oct 9, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.84.3](0.84.2...0.84.3)
(2023-10-09)


### Features

* regular replace package dependency and replace package with
no-main-branch
([#1481](#1481))
([bec49ac](bec49ac))
* remove logs on enclave rm and clean -a
([#1489](#1489))
([9ea344e](9ea344e))


### Bug Fixes

* Add new line while inspecting file contents
([#1477](#1477))
([545aa53](545aa53))
* improve absolute locator checks
([#1498](#1498))
([cda001d](cda001d))
* kurtosis web cmd work for remote context
([#1486](#1486))
([8d8634c](8d8634c))
* make vector use ISO week time
([#1497](#1497))
([e6d1f5e](e6d1f5e))
* replace duplicate log files with symlinks
([#1472](#1472))
([57da901](57da901))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: h4ck3rk3y <h4ck3rk3y@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The files artifact content display does not preserve newlines File Artifacts Api does not return new lines
2 participants