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

Suppress comments from BenchmarkCI #741

Merged
merged 1 commit into from Feb 28, 2020
Merged

Suppress comments from BenchmarkCI #741

merged 1 commit into from Feb 28, 2020

Conversation

tkf
Copy link
Member

@tkf tkf commented Feb 19, 2020

See: #737 (comment)

It's good to go (if this setup is desired) but I submit it as a draft to avoid accidentally merging this.

@c42f
Copy link
Member

c42f commented Feb 20, 2020

Thanks! Let's see how the existing config goes for a bit and we can merge this if it seems necessary.

@c42f
Copy link
Member

c42f commented Feb 28, 2020

I'm going to merge this to cut down on the number of notifications.

The HTML version is kinda nice, but the text version in the job logs is just as good I think.

@c42f c42f marked this pull request as ready for review February 28, 2020 01:01
@c42f c42f merged commit c808bdd into master Feb 28, 2020
@c42f c42f deleted the tkf/no-comments branch February 28, 2020 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants