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.
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
Another attempt at an astable flag #298
Changes from 25 commits
a8701c8
9b997a6
d639560
b77e8ca
3cdf0d5
b878fbb
2344a2e
6557def
6002def
08a1c4b
581b2cf
7cc8947
0eca67d
a4ab9a6
ab9bae4
495f08a
01cb5e7
01fb3b7
915191c
a331fc2
2ce4d9e
57b4051
da7674d
285e3ac
713eaf0
4e01c4a
09c692a
ae26da8
a7fd1a2
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.