Skip to content
This repository was archived by the owner on May 23, 2024. It is now read-only.

Add up to 6 ICEs #351

Merged
merged 3 commits into from
May 6, 2020
Merged

Add up to 6 ICEs #351

merged 3 commits into from
May 6, 2020

Conversation

fanninpm
Copy link
Contributor

@fanninpm fanninpm commented May 4, 2020

71546 might need some work done on the main() function.

@JohnTitor
Copy link
Member

Overall looks good, could you drop 71805.rs since the issue has been fixed already and add a new line to the end of each file?

@JohnTitor
Copy link
Member

Also needs a rebase.

@fanninpm
Copy link
Contributor Author

fanninpm commented May 6, 2020

Do I need to squash commits?

@JohnTitor
Copy link
Member

Do I need to squash commits?

I'd prefer so.

Add trailing newlines and move 71805 to fixed
@fanninpm
Copy link
Contributor Author

fanninpm commented May 6, 2020

Are 70934 and 71798 fixed as well?

@JohnTitor
Copy link
Member

JohnTitor commented May 6, 2020

Oops, sorry! I mistook 71798 for 71805, the later hasn't been fixed yet. 70934 is also fixed indeed.

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@JohnTitor JohnTitor merged commit 9a80233 into rust-lang:master May 6, 2020
@fanninpm fanninpm deleted the add-ices branch May 6, 2020 16:22
@fanninpm
Copy link
Contributor Author

fanninpm commented May 6, 2020

Glad to help! Where do I go for further questions unrelated to this PR?

@JohnTitor
Copy link
Member

Where do I go for further questions unrelated to this PR?

If it's related to glacier, the issue tracker is one of the good places. If related to Rust itself, then we have some places, e.g. Discord, Discourse, Zulip, etc.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants