-
Notifications
You must be signed in to change notification settings - Fork 55
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
Another attempt at an astable flag #298
Merged
Merged
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
a8701c8
initial attempt
pdeffebach 9b997a6
finally working
pdeffebach d639560
start adding tests
pdeffebach b77e8ca
more tests
pdeffebach 3cdf0d5
more tests
pdeffebach b878fbb
add docstring
pdeffebach 2344a2e
tests pass
pdeffebach 6557def
add ByRow in docstring
pdeffebach 6002def
add type annotation
pdeffebach 08a1c4b
better docs
pdeffebach 581b2cf
more docs fixes
pdeffebach 7cc8947
update index.md
pdeffebach 0eca67d
Apply suggestions from code review
pdeffebach a4ab9a6
Merge branch 'astable_2' of https://github.com/pdeffebach/DataFramesM…
pdeffebach ab9bae4
clean named tuple creation
pdeffebach 495f08a
add example with string
pdeffebach 01cb5e7
grouping tests
pdeffebach 01fb3b7
Update src/macros.jl
pdeffebach 915191c
changes
pdeffebach a331fc2
Merge branch 'astable_2' of https://github.com/pdeffebach/DataFramesM…
pdeffebach 2ce4d9e
fix some errors
pdeffebach 57b4051
add macro check
pdeffebach da7674d
add errors for bad flag combo
pdeffebach 285e3ac
better grouping tests
pdeffebach 713eaf0
Update src/parsing_astable.jl
pdeffebach 4e01c4a
add snipper to transform, select, combine, by
pdeffebach 09c692a
add mutating tests
pdeffebach ae26da8
get rid of debugging printin
pdeffebach a7fd1a2
Apply suggestions from code review
pdeffebach File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While we are at it given our recent discussion on Discourse, I think it is essential to mention when the
$
reference is resolved.Also maybe add an example when macros are used within a function? I think these are cases not trivial. This can be another PR of course
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will do this as another PR. In summary, you can't use other macros which use
$
. I will try and sort out if I can carve out an exception.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be clear why I stress it so much. With DataFrames.jl my answer to users is: if you learn Julia Base then you will know exactly how DataFrames.jl works. With DataFramesMeta.jl unfortunately this is not the case as it is a DSL so we need to be very precise how things work in documentation.