Skip to content
This repository has been archived by the owner on Feb 8, 2022. It is now read-only.

fix Enum in message body error #96

Merged
merged 3 commits into from Oct 9, 2018
Merged

Conversation

7sharp9
Copy link
Collaborator

@7sharp9 7sharp9 commented Oct 8, 2018

fix #93


[<Fact>]
let ``proto3 message with enum type doesn't parse (#93)`` () =
System.Diagnostics.Debugger.Break()
Copy link
Owner

Choose a reason for hiding this comment

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

Guessing this line isn't needed

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oops, yeah, removed :-)

@ctaggart ctaggart changed the title Fix for #93 fix Enum in message body error Oct 9, 2018
@ctaggart ctaggart added this to the 0.6.1 milestone Oct 9, 2018
@ctaggart ctaggart merged commit 40a86c7 into ctaggart:master Oct 9, 2018
@ctaggart ctaggart modified the milestones: 0.6.1, 0.6.2 Oct 9, 2018
@ctaggart
Copy link
Owner

ctaggart commented Oct 9, 2018

published 0.6.2 to NuGet Gallery

ctaggart added a commit that referenced this pull request Mar 19, 2019
@ctaggart
Copy link
Owner

So the tests actually failed for this pull request in Froto.Parser.Test, so I'm going to git revert 40a86c7 and reopen #93. Unfortunately, 0.6.2 Froto.Parser is broken according to the tests. The AppVeyor build should have failed when the tests failed, but didn't. I'm revising some build stuff in a new branch and just noticed this failure. @7sharp9

ctaggart added a commit that referenced this pull request Mar 19, 2019
* add Nerdbank.GitVersioning

* temp remove SourceLink

* try switching Froto.Parser.Test from netsoreapp2.0 to 2.1

* use Froto.sln

* latest dotnet 2.2.3

* try 2.2.2

* sdk is 2.2.105

* dist: xenial

* remove Froto.TypeProvider from Froto.sln

* back to netcoreapp2.0

* git fetch --unshallow

* Revert "fix Enum in message body error (#96)"

This reverts commit 40a86c7.

* add version.json

* disable nbgv for exe

* use previous SourceLink
dotnet/sourcelink#236

* PrivateAssets="All"

* upgrade to Nerdbank.GitVersioning" Version="2.3.136

* remove Break()
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.

Enum in message body error
2 participants