Skip to content

Remove duplicated line #559

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

Merged
merged 1 commit into from
Dec 15, 2015
Merged

Remove duplicated line #559

merged 1 commit into from
Dec 15, 2015

Conversation

pahmed
Copy link
Contributor

@pahmed pahmed commented Dec 15, 2015

No description provided.

gottesmm added a commit that referenced this pull request Dec 15, 2015
Remove duplicated comment line from test.
@gottesmm gottesmm merged commit da5ef15 into swiftlang:master Dec 15, 2015
@pahmed pahmed deleted the patch-1 branch December 15, 2015 17:53
@shahmishal
Copy link
Member

Reverting this change due to test failure "IDE/dump_api.swift"

@pahmed
Copy link
Contributor Author

pahmed commented Dec 16, 2015

Sorry, but how removing this comment could cause test failure?

@rudkx
Copy link
Contributor

rudkx commented Dec 16, 2015

@pahmed The file you changed is the expected test output. The test input (dump_api.swift) has this comment in two different places, and apparently the dumper is emitting them next to one another in the output.

@pahmed
Copy link
Contributor Author

pahmed commented Dec 17, 2015

Got it, thanks rudkx

freak4pc pushed a commit to freak4pc/swift that referenced this pull request Sep 28, 2022
[Stress tester XFails] Update XFails
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants