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

Nuget setup #9

Merged
merged 4 commits into from
Aug 9, 2016
Merged

Nuget setup #9

merged 4 commits into from
Aug 9, 2016

Conversation

marrony
Copy link
Contributor

@marrony marrony commented Aug 4, 2016

No description provided.


See https://faunadb.com for more information.</description>
<licenseUrl>https://github.com/faunadb/faunadb-csharp/blob/master/LICENSE</licenseUrl>
<projectUrl>https://faunadb.com/</projectUrl>
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't the projectUrl be the github url?

@erickpintor
Copy link
Contributor

lgtm

@@ -28,7 +28,7 @@ First install the Nuget package by editing your `packages.config`
```xml
<?xml version="1.0" encoding="utf-8"?>
<packages>
<package id="FaunaDB" version="1.0.0" targetFramework="net45" />
<package id="FaunaDB.Client" version="0.0.1" targetFramework="net45" />
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't this be 0.1.0, not 0.0.1

@erickpintor
Copy link
Contributor

🐑 it!

@@ -0,0 +1,3 @@
Remove-Item packages\FaunaDB.Client -Recurse -Force
Make-Item packages\FaunaDB.Client
Copy link
Contributor

Choose a reason for hiding this comment

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

its New-Item i think

Copy link
Contributor

Choose a reason for hiding this comment

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

New-Item -ItemType directory iirc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeap, now it's tested on windows

@erickpintor
Copy link
Contributor

Is there anything missing here? Can we ship it and release C#?

@marrony
Copy link
Contributor Author

marrony commented Aug 9, 2016

Nothing that I'm aware of

@erickpintor
Copy link
Contributor

@ed Is this PR ok for you?

@eaceaser
Copy link
Contributor

eaceaser commented Aug 9, 2016

Yeah this is good

@eaceaser
Copy link
Contributor

eaceaser commented Aug 9, 2016

ship

@erickpintor erickpintor merged commit b6c779c into master Aug 9, 2016
@erickpintor erickpintor deleted the nuget-publish branch August 9, 2016 18:03
parkhomenko pushed a commit that referenced this pull request Oct 22, 2021
Changed query constructors to use varargs where possible
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.

None yet

3 participants