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

ARROW-9845: [Rust] [Parquet] Move serde_json dependency to dev-dependencies as it is only used in tests #8087

Closed
wants to merge 1 commit into from

Conversation

alamb
Copy link
Contributor

@alamb alamb commented Aug 31, 2020

This PR is a cherry pick of f2e79ca by @saethlin against latest master (to get a clean CI run) as suggested #8042 (comment)

@github-actions
Copy link

@alamb alamb force-pushed the alamb/ARROW-9845-dependencies branch from 35093af to e99754b Compare September 1, 2020 10:15
@alamb alamb changed the title ARROW-9845: [Rust] [Parquet] Rebased serde_json is only used in tests but isn't in dev-dependencies ARROW-9845: [Rust] [Parquet] Move serde_json dependency to dev-dependencies as it is only used in tests Sep 1, 2020
@andygrove andygrove closed this in 2a0fc0a Sep 2, 2020
emkornfield pushed a commit to emkornfield/arrow that referenced this pull request Oct 16, 2020
…encies as it is only used in tests

This PR is a cherry pick of f2e79ca by @saethlin against latest master (to get a clean CI run) as suggested apache#8042 (comment)

Closes apache#8087 from alamb/alamb/ARROW-9845-dependencies

Authored-by: Ben Kimock <kimockb@gmail.com>
Signed-off-by: Andy Grove <andygrove73@gmail.com>
@alamb alamb deleted the alamb/ARROW-9845-dependencies branch October 26, 2020 20:04
GeorgeAp pushed a commit to sirensolutions/arrow that referenced this pull request Jun 7, 2021
…encies as it is only used in tests

This PR is a cherry pick of f2e79ca by @saethlin against latest master (to get a clean CI run) as suggested apache#8042 (comment)

Closes apache#8087 from alamb/alamb/ARROW-9845-dependencies

Authored-by: Ben Kimock <kimockb@gmail.com>
Signed-off-by: Andy Grove <andygrove73@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants