Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Add GoString for Value

Đang mở
#82 0 bình luận 0 reaction 1 người được giao Xem trên GitHub

@f2prateek đang làm issue này rồi.

Từ ngày 10/4/2018.

Đánh giá

Issue này chưa được đánh giá.

Mô tả

This came up recently when I was trying to test some code that calls the stats package like this: eng.Add("foo", uint64(1)).

// test

handler := &statstest.Handler{}
eng := stats.NewEngine("test", handler)

runCode(eng)

assert.Equal(t, []stats.Measure{
{
            Name: "test.foo",
            Fields: []stats.Field{
                stats.MakeField("", 1, stats.Counter),
            },
            Tags: []stats.Tag{},
        }
}, handler.Measures())

This test failed with a cryptic error: assert.go:24: ! [0].Fields[0].Value.typ: 2 != 3.

The error was that my code is using uint, but the test is using an `int.

If the value type had a GoString method, the error could have been more obvious.

Ngôn ngữ chính
Go
Star
215
Fork
31
Merge trung bình
8 phút
Pull request đã merge (30 ngày)
2

Chuẩn bị môi trường

  • Có Dockerfile hoặc tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của segmentio/stats

Tất cả issue của segmentio/stats

Issue tương tự

Thêm issue về Go

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.