Skip to content
This repository has been archived by the owner on Oct 31, 2021. It is now read-only.

Conversion to Paket #12

Merged
merged 9 commits into from
Jul 22, 2015
Merged

Conversation

forki
Copy link
Collaborator

@forki forki commented Jan 6, 2015

Are you interested in this?

@dmitry-a-morozov
Copy link
Contributor

@forki Please, give me some time to think it through.
I have 3 concerns:

  1. Paket is yet to reach maturity level
  2. I understand problems Paket tries to solve but I don't have those for this library because dependencies are trivial.
  3. I don't have any Paket experience so there will be some learning curve. I don't think it will be hard but I have gazillion other things to do.

That said I admire and support your effort so I'm willing to take this relatively small risk. Give me couple days to decide.

P.S. Have RProvider and Deedle been converted to Paket?

@forki
Copy link
Collaborator Author

forki commented Jan 7, 2015

Yes I totally understand.

Deedle pull request is still open
fslaborg/Deedle#288
On Jan 7, 2015 6:43 AM, "Dmitry Morozov" notifications@github.com wrote:

@forki https://github.com/forki Please, give me some time to think it
through.
I have 3 concerns:

  1. Paket is yet to reach maturity level
  2. I understand problem Paket tries to solve but I don't have those for
    this library because dependencies are trivial.
  3. I don't have any Paket experience so there will be some learning curve.
    I don't think it will be hard but I have gazillion other things to do.

That said I admire and support your effort so I'm willing to take this
relatively small risk. Give me couple days to decide.

P.S. Have RProvider and Deedle been converted to Paket?


Reply to this email directly or view it on GitHub
#12 (comment)
.

@christolo
Copy link

Any update on this? Paket would be nice to have.

@dmitry-a-morozov
Copy link
Contributor

@christolo Can you review this PR making sure it can be merged?
Once it's done I will merge.

@christolo
Copy link

Sure. Not sure of the best way to handle this though. Try to merge across forks? Or just fork and replay @forki changes?

@dmitry-a-morozov
Copy link
Contributor

I'm not sure either.
Probably fork and reply will work.

@forki
Copy link
Collaborator Author

forki commented Jul 22, 2015

I can try to cleanup this.

@forki
Copy link
Collaborator Author

forki commented Jul 22, 2015

I updated the PR, but it#s not building. Any ideas?

@sergey-tihon
Copy link
Contributor

Could you try build it using AppVeyor?
I think that it is Mono specific issue. Probably PresentationFramework.dll differ from Windows one.

@forki
Copy link
Collaborator Author

forki commented Jul 22, 2015

dmitry-a-morozov added a commit that referenced this pull request Jul 22, 2015
@dmitry-a-morozov dmitry-a-morozov merged commit 80f6074 into fsprojects-archive:master Jul 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants