Use \newline instead of \\ for linebreak (#2)

- use \newline latex command instead of \\
- increase version number
- upgrade to Go 1.21
- add token to codecov action

.github/workflows/ci.yml

@@ -16,7 +16,7 @@ jobs:
  - name: Install go
  uses: actions/setup-go@v3
  with:
- go-version: '^1.20.0'
+ go-version: '^1.21.0'
  - name: "Installed go version"
  run: go version
  - name: "Build"
@@ -26,4 +26,7 @@ jobs:
  run: |
  go test ./... -race -coverprofile=coverage.out
  - name: "Code coverage"
- uses: codecov/codecov-action@v3
+ uses: codecov/codecov-action@v3
+ with:
+ token: ${{ secrets.CODECOV_TOKEN }}
+ fail_ci_if_error: true

.github/workflows/release.yml

@@ -2,7 +2,7 @@ on:
  workflow_dispatch
 
 env:
- MAJOR_MINOR_PATCH: 0.1.0
+ MAJOR_MINOR_PATCH: 0.2.0
  FEATURE_BRANCH_BUILD: 1
 
 concurrency: release-${{ github.ref }}

encoding/protolua/encode_test.go

@@ -229,14 +229,14 @@ func TestMarshal_Positive(t *testing.T) {
  input: &v3proto.SingleString{
  Fut: "\n",
  },
- expected: "SingleString={fut=\"\\\\\\\\\"}",
+ expected: "SingleString={fut=\" \\\\newline \"}",
  },
  {
  desc: "Message with special character carriage return and line feed",
  input: &v3proto.SingleString{
  Fut: "\r\n",
  },
- expected: "SingleString={fut=\"\\\\\\\\\"}",
+ expected: "SingleString={fut=\" \\\\newline \"}",
  },
  {
  desc: "Message with special character \\",
@@ -362,7 +362,7 @@ func TestMarshal_GoogleProtobuf_CanBeOverridden(t *testing.T) {
 
  actual, err := LuaMarshalOption{AdditionalMarshalers: []interface {
  Handle(fullName protoreflect.FullName) (MarshalFunc, error)
- }{ &TestOverrider{}}}.marshal(message)
+ }{&TestOverrider{}}}.marshal(message)
 
  AssertLuaString(actual, err, expected, t)
 }
@@ -387,8 +387,8 @@ func (TestOverrider) Handle(fullName protoreflect.FullName) (MarshalFunc, error)
 
  convertTimestamp := func(encodingRun EncodingRun, m protoreflect.Message) error {
  err := encodingRun.Encoder.WriteString("FortyTwo")
- if(err != nil) {
- return err;
+ if err != nil {
+ return err
  }
  return nil
  }
@@ -403,4 +403,3 @@ func (TestOverrider) Handle(fullName protoreflect.FullName) (MarshalFunc, error)
  }
  return nil, nil
 }
-

encoding/protolua/encoder.go

@@ -210,7 +210,7 @@ func appendString(out []byte, in string) ([]byte, error) {
  case '\f':
  return out, errors.New("not implemented yet")
  case '\n': // replaced with a latex line break that needs to be escaped
- out = append(out, "\\\\\\\\"...)
+ out = append(out, " \\\\newline "...)
  case '\r':
  // do nothing as \r\n and \n are reduced to line break
  case '\t':

go.mod

@@ -1,6 +1,6 @@
 module github.com/kinneko-de/protobuf-go
 
-go 1.20
+go 1.21
 
 require (
  github.com/google/go-cmp v0.5.9