Skip to content

Remove FIXME referring to a closed issue (#3654) #13380

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

Closed
wants to merge 2 commits into from
Closed

Remove FIXME referring to a closed issue (#3654) #13380

wants to merge 2 commits into from

Conversation

smesseim
Copy link
Contributor

@smesseim smesseim commented Apr 7, 2014

Issue #3654 had already been fixed and closed, but the FIXME comment has
not been removed. This commit removes the FIXME comment.

smesseim added 2 commits April 7, 2014 17:41
Issue #3654 had already been fixed and closed, but the FIXME comment has
not been removed. This commit removes the FIXME comment.
The years of copyright and the name of the copyright holder were not
present in the notice.

The Apache license was added to the project in 2012, so 2012 is the
starting year. 2014 is the current year. The copyright holder is the
Mozilla Foundation (taken from the MIT license).
@alexcrichton
Copy link
Member

This PR appears to be confusing bors, I'm going to temporarily close to see if it makes bors happier.

@alexcrichton
Copy link
Member

I'm a little unsure why, but PR seems to upset bors (sorry!).

Would you mind reopening with another PR? This also appears to have picked up a commit from #13391 by accident.

bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 18, 2022
Honor cfg attributes on params when lowering their patterns

Closes rust-lang/rust-analyzer#13375
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 24, 2024
Check that #[deny(allow_attributes)] do not issue spurious messages

Add a new test for rust-lang#13380. This bug was caused by a bug in rustc which has been fixed in rust-lang@6ee87ae.

Close rust-lang#13380

changelog: [`allow_attributes`]: fix spurious warning messages
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.

2 participants