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

Test for newlines in omero obj #6265

Merged
merged 2 commits into from
Mar 15, 2021
Merged

Conversation

joshmoore
Copy link
Member

see: ome/omero-py#279

Without this PR, no attempt to pass a newline to the obj command will succeed. e.g.:

omero obj update Dataset:123 description=$'A\nB'

or

omero obj update Dataset:123 description="A

B"

will stop at the A.

With this PR, both usages (and likely others) will correctly pass a newline to the server.

@joshmoore joshmoore merged commit f19058e into ome:develop Mar 15, 2021
@joshmoore joshmoore deleted the newline-test branch March 15, 2021 11:29
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.

3 participants