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

internal/x/text/unicode/bidi: ignore unused return values in test #30015

Conversation

leonklingele
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes Used by googlebot to label PRs as having a valid CLA. The text of this label should not change. label Jan 30, 2019
@gopherbot
Copy link

This PR (HEAD: 7ffd936) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/160439 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@leonklingele leonklingele force-pushed the internal-x-text-unicode-bidi-ignore-unused-return-values branch from 7ffd936 to bb0ebc9 Compare February 4, 2019 12:34
@gopherbot
Copy link

This PR (HEAD: bb0ebc9) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/160439 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@leonklingele leonklingele changed the title internal/x/text/unicode/bidi: ignore unused return values internal/x/text/unicode/bidi: ignore unused return values in test Feb 4, 2019
@gopherbot
Copy link

Message from Emmanuel Odeke:

Patch Set 3: Code-Review-1

Thank you Leon for sending this CL!

Unfortunately the change you've made is on an auto-generated file.
Please see line 1 of that file which says:

// Code generated by running "go run gen.go -core" in golang.org/x/text. DO NOT EDIT.


Please don’t reply on this GitHub thread. Visit golang.org/cl/160439.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link

Message from Leon Klingele:

Patch Set 3:

Is that test file even used for anything? Grepping my whole GOPATH for "StringAttributes" (a type used in example_test.go) doesn't reveal where exactly the type is defined. (Also see https://go-review.googlesource.com/c/go/+/160445/1/src/internal/x/text/unicode/bidi/example_test.go#87 for why the file likely will not compile).


Please don’t reply on this GitHub thread. Visit golang.org/cl/160439.
After addressing review feedback, remember to publish your drafts!

@leonklingele
Copy link
Contributor Author

File should be removed altogether. #30671

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Used by googlebot to label PRs as having a valid CLA. The text of this label should not change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants