Skip to content

Fix test after several PR were merged #315

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Mar 28, 2021
Merged

Conversation

dkm
Copy link
Member

@dkm dkm commented Mar 27, 2021

the PR #302 and #301 were conflicting because they are both modifying the tests.
Move the conflicting test and add correct dg-error directive.

@dkm
Copy link
Member Author

dkm commented Mar 27, 2021

I'll test this with an updated tree and check the tests are correct

the PR Rust-GCC#302 and Rust-GCC#301 were conflicting because they are both modifying the tests.
Move the conflicting test and add correct dg-error directive.

continue1.rs is less verbose after Rust-GCC#286, remove the dg-excess-error.
@dkm
Copy link
Member Author

dkm commented Mar 27, 2021

                === rust Summary ===

# of expected passes            793
# of expected failures          26

Also fixed the continue1.rs test.

@philberty
Copy link
Member

Nice one @dkm do you have the ability to merge now? If so you should give it a go :)

@dkm
Copy link
Member Author

dkm commented Mar 27, 2021

Nice one @dkm do you have the ability to merge now? If so you should give it a go :)

I don't seem to have this kind of power :)

@dkm dkm merged commit cf10b02 into Rust-GCC:master Mar 28, 2021
@dkm
Copy link
Member Author

dkm commented Mar 28, 2021

Nice one @dkm do you have the ability to merge now? If so you should give it a go :)

I don't seem to have this kind of power :)

Scratch that, I have permission to merge... Not sure if you added them or if I missed it yesterday :)

@dkm dkm deleted the pr/fix_test branch March 28, 2021 09:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants