search

queryverse / Query.jl

Query almost anything in julia
Other
396 stars 50 forks source link

Add @select, @rename and @mutate macros #209

Closed davidanthoff closed 5 years ago

davidanthoff commented 6 years ago

Fixes #208, fixes #203 and fixes #202.

codecov-io commented 6 years ago

Codecov Report

Merging #209 into master will decrease coverage by 9.52%. The diff coverage is 97.05%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #209      +/-   ##
=========================================
- Coverage   92.22%   82.7%   -9.53%     
=========================================
  Files           5       6       +1     
  Lines         296     399     +103     
=========================================
+ Hits          273     330      +57     
- Misses         23      69      +46
Impacted Files Coverage Δ
src/Query.jl 76.92% <ø> (ø) :arrow_up:
src/Query_macros.jl 97.05% <97.05%> (ø)
src/sinks/sink_type.jl 50% <0%> (-50%) :arrow_down:
src/query_translation.jl 80.89% <0%> (-11.7%) :arrow_down:
src/standalone_query_macros.jl 100% <0%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ec9be56...a32da52. Read the comment docs.

TonyLianLong commented 5 years ago

Now that @select has been moved to QueryOperators.jl, probably we should change the Query.@select in the tests to QueryOperators.@select?

Yijia-Chen commented 5 years ago

All the helper functions are now in NamedTupleUtilities.jl and all tests pass. The pull request should be able to merge.