액세서리를 통한 세트 설정 허용

묘사

목표는 #112 종료입니다.
나는 Benchmark::Suite 에 대한 인용을 삭제했다. 나는 어디에서도 찾을 수 없다
그것은 사용되었고, 최초의 제출은 아주 오래 전으로 거슬러 올라갈 수 있다.보아하니
마치git에서 찾을 수 없는 클래스나 모듈의 유적지 같아요.
역사적 경유

git grep 'Suite' $(git rev-list --all)

모든 구성이 job.config job_opts에 전달되고 이제 블록은
이전 평가에서는 덮어쓰기를 허용했습니다.

토론 #1

이거 괜찮아 보여요.
비록 set_reporter 방법은 싫지만,if문장을 삭제하면 확실히 그것을 더욱 좋아지게 할 수 있다.

토론 #2

@kbrock 댓글 감사합니다!

토론 #셋

이거 괜찮아 보여요.
나는 네가 갈 곳을 좋아한다reporter=하지만 제가 여기에 약간의 PRs를 기여했지만 저는 관리자가 아니기 때문에 제 말에 대해 유보적인 태도를 가져주세요.
출력을 더 잘 포착하기 위해 PRs를 넣었는데, 이것이 바로 내가 여기서 소리를 내는 이유다.많은 사람들이 더 좋은 생산을 원한다고 말한다. 그래서 나는 다른 홍보 회사를 개설했다.이것은 약간 중첩된 것 같으니 계속해 주십시오. 나는 이것을 재구성할 것입니다.

토론 #4

헤이@allcentury, 기초를 다시 설정하고 당신의 약속을 압축하는 방법을 아십니까?
이것은 미래의git역사를 더욱 분명하게 할 것이다.(지금의 변화는 미래의 역사...마티 맥플리가 자랑스러워할 것이다)

토론 #5

헤이@allcentury-더 보충할 것이 있습니까, 아니면 제 입력을 기다리고 있습니까?

토론 #6

@kbrock- 네, 저는 fixup 흐름을 따라 합병을 준비하고 압박할 때까지 따라갑니다.
@nateberkopec-감사합니다!네, 당신의 의견을 이용할 수 있습니다.
옵션 A:

는 반대quiet하고 설치reporter에 찬성했다.

출력이 필요하지 않으면 NoopReporter를 사용합니다.

quiet의 문서를 NoopReporter

로 교체

최종적으로 누구든지 자기Reporter를 주사할 수 있도록 허락한다interface

format+reporter를 합병합니까?CsvReporter, JsonReporter 등

옵션 B:

보류quiet 떠오르지 마라reporter 생각

반대/변경 필요 없음

형식 구성과 출력 구성

을 병합하지 마십시오.
처음에 나는 reporter 사용자가 자신의 포맷 프로그램을 작성할 수 있다고 생각했다. 예를 들어 csv, json 등이다. 그러나 나는 format option를 보았다.현재 상황에서 이와 공공 관계는 다음과 같다.StdoutReport :

모든 내용을 stdout

에 전송NoopReport :

출력 없음

:format

원시 또는 인류는 상술한 보고서류에 사용된다.

당신은 사용자로 하여금 자신의Report류를 주입하게 하는 좋은 이유를 생각해 낼 수 있습니까?나는 형식을 병합하지 않을 수 없다.그래서 나는 지금 B를 선택하는 경향이 있다.사상

토론 #7

Can you think of a good reason to have users inject their own Report class? I can't without conflating format. So I'm leaning towards Option B now. Thoughts?

부탁이 하나 있어요.https://github.com/evanphx/benchmark-ips/issues/81출력을 포획할 수 있다.따라서 문자열 출력 버퍼나 다른 것을 설정할 수 있다면 이gem는 이 문제를 해결하는 방향으로 발전할 것이다.나는 이 문제를 해결하기 위해 홍보를 시작했지만, 나는 이 홍보의 발전 방향을 좋아한다. 네가 완성한 후에 다른 홍보를 복원할 수 있다.

토론 #8

응, 벗어나고 싶지 않아quiet.나는 배경quiet을 배경에 있는 기자를 조용히 바꿔야 한다고 생각한다.그래서 선택을 하게 된 것 같아요. B?나는 공관 중의 기타 결정/요점은 오늘 해결할 필요가 없다고 생각한다. 아마도 우리는 장래에 기자/정보 발표자를 합병할 것이다. 그러나 나는 지금 이 문제를 해결할 필요가 없다고 생각한다.

토론 #9

@nateberkopec-감사합니다.나는 이것이 이미 너희들의 심사를 제공할 준비가 되어 있다고 생각한다.

토론 #10

그 방법 말고는 뭐든지 다 좋아해요.
그것은 단지 한 장소에서만 사용하기 때문에, 이것은 간단한 변화인 것 같다
하지만 분명히 나는 이 점에서 닉

토론 #11

1의 이름을 따랐다. 이것은GTG

토론 #12

@nateberkopec의 완벽함이다. 고마워!

토론 #13

를 삭제했습니다. 현재set는 reporter에 의존하는 순수 함수입니다.감사합니다 @allcentury quiet= 감사합니다 @allcentury