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

PR 16150 use Symbol(x...) and deprecate symbol(x...) #16150

Closed
wants to merge 1 commit into from
Closed

PR 16150 use Symbol(x...) and deprecate symbol(x...) #16150

wants to merge 1 commit into from

Conversation

jeffreysarnoff-dev
Copy link

@jeffreysarnoff-dev jeffreysarnoff-dev commented May 1, 2016

picking up detached github branch for the automated checking (from #16130)

@@ -1082,6 +1082,9 @@ end
@deprecate sprand{T}(n::Integer, density::AbstractFloat, ::Type{T}) sprand(T, n, density)
@deprecate sprand{T}(r::AbstractRNG, n::Integer, density::AbstractFloat, ::Type{T}) sprand(r, T, n, density)

# #16130
@deprecated symbol Symbol
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deprecate, not @deprecated

@yuyichao
Copy link
Contributor

yuyichao commented May 1, 2016

Could you please try harder to force push to the branch you are working on?
A new PR every time you update something is starting to be annoy...

You can practice on a different branch with possibly a PR on your fork if you are not sure what you are doing.

@tkelman
Copy link
Contributor

tkelman commented May 1, 2016

All the past PR's now say "unknown repository" which I think can only happen if you delete your fork. That's really not necessary. The persistence is appreciated, but please ask for help with git rather than making more of a mess with multiple PR's.

@jeffreysarnoff-dev jeffreysarnoff-dev changed the title PR 16130 use Symbol(x...) and deprecate symbol(x...) PR 16150 use Symbol(x...) and deprecate symbol(x...) May 1, 2016
@jeffreysarnoff-dev
Copy link
Author

throwing in the towel

@tkelman
Copy link
Contributor

tkelman commented May 1, 2016

We're a couple lines away this time. One of us can take over and finish the final few lines, though that would be a sixth PR for this. Unless you'd like to give one of us collaborator access to your fork and we could reopen and finish this for you?

@hayd
Copy link
Member

hayd commented May 1, 2016

I have fixed up a branch. Let me open a PR.

@hayd
Copy link
Member

hayd commented May 1, 2016

It looks like all of the above comments were already addressed.

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.

5 participants