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

ENG-3044: Introduce set streaming support #184

Merged
merged 2 commits into from
Feb 8, 2022
Merged

Conversation

erickpintor
Copy link
Contributor

Changes:

  • Adding missing event field.

faunaee
faunaee previously approved these changes Jan 31, 2022
Copy link
Contributor

@faunaee faunaee left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@@ -53,6 +50,7 @@ commands:
- run:
name: Run Tests 5.0
command: |
apt-get install -y dotnet-sdk-5.0

Choose a reason for hiding this comment

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

There's also a version 5 SDK image available, so you could remove this line and put this on line 12 above:

- image: mcr.microsoft.com/dotnet/sdk:5.0

Choose a reason for hiding this comment

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

Since this passes, it's probably not necessary, just another option.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah cool, was thinking maybe we keep it as is and start running tests on dotnet 6.

henryfauna
henryfauna previously approved these changes Feb 8, 2022
@cleve-fauna cleve-fauna merged commit 54081d0 into main Feb 8, 2022
@cleve-fauna cleve-fauna deleted the set-streaming-support branch February 8, 2022 16:50
cleve-fauna added a commit that referenced this pull request Feb 8, 2022
* introduce set streaming support

* Fix circleci config - test dotnet 5.0 correctly, use public FaunaDB image

Co-authored-by: Cleve Stuart <[email protected]>
@cleve-fauna cleve-fauna mentioned this pull request Feb 8, 2022
macnealefauna pushed a commit that referenced this pull request Feb 8, 2022
* OSS-775: Create docs for C# driver (#166)

* ENG-3044: Introduce set streaming support (#184)

* introduce set streaming support

* Fix circleci config - test dotnet 5.0 correctly, use public FaunaDB image

Co-authored-by: Cleve Stuart <[email protected]>

Co-authored-by: shestakovg <[email protected]>
Co-authored-by: Erick Pintor <[email protected]>
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

5 participants