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

Don't use ExpnKind::descr to get the name of a bang macro. #5096

Merged
merged 1 commit into from
Jan 26, 2020

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Jan 26, 2020

This is the same change as the first commit in rust-lang/rust#68407, but applied to clippy.
The new code should work both before and after the changes in rust-lang/rust#68407.

changelog: none

@oli-obk
Copy link
Contributor

oli-obk commented Jan 26, 2020

@bors r+

@bors
Copy link
Contributor

bors commented Jan 26, 2020

📌 Commit 0e52a0e has been approved by oli-obk

bors added a commit that referenced this pull request Jan 26, 2020
Don't use ExpnKind::descr to get the name of a bang macro.

This is the same change as the first commit in rust-lang/rust#68407, but applied to clippy.
The new code should work both before and after the changes in rust-lang/rust#68407.
@bors
Copy link
Contributor

bors commented Jan 26, 2020

⌛ Testing commit 0e52a0e with merge f690786...

@bors
Copy link
Contributor

bors commented Jan 26, 2020

💔 Test failed - checks-travis

@eddyb
Copy link
Member Author

eddyb commented Jan 26, 2020

ERROR: PR body must contain 'changelog: ...'

Huh, how come that's not checked in the PR CI?

@eddyb
Copy link
Member Author

eddyb commented Jan 26, 2020

@bors retry

@bors
Copy link
Contributor

bors commented Jan 26, 2020

@eddyb: 🔑 Insufficient privileges: not in try users

@llogiq
Copy link
Contributor

llogiq commented Jan 26, 2020

@bors retry

@bors
Copy link
Contributor

bors commented Jan 26, 2020

⌛ Testing commit 0e52a0e with merge b6a9766...

bors added a commit that referenced this pull request Jan 26, 2020
Don't use ExpnKind::descr to get the name of a bang macro.

This is the same change as the first commit in rust-lang/rust#68407, but applied to clippy.
The new code should work both before and after the changes in rust-lang/rust#68407.

changelog: none
@bors
Copy link
Contributor

bors commented Jan 26, 2020

☀️ Test successful - checks-travis, status-appveyor
Approved by: oli-obk
Pushing b6a9766 to master...

@bors bors merged commit 0e52a0e into rust-lang:master Jan 26, 2020
@eddyb eddyb deleted the mac-name branch January 26, 2020 10:16
@mati865
Copy link
Contributor

mati865 commented Jan 26, 2020

Huh, how come that's not checked in the PR CI?

Unfortunately that's not an easy task, you see links in #4211

@flip1995
Copy link
Member

Closing this, since

  1. Secret env vars don't work on PRs from forks (obv security reasons)
  2. Pulling the PR body through the github API doesn't work, since we need some kind of API token. And using this API token is (AFAIK) only possible through secrets.

@eddyb
Copy link
Member Author

eddyb commented Jan 27, 2020

@flip1995 would moving to github actions fully fix this?

@flip1995
Copy link
Member

Maybe. Depends if the GITHUB_TOKEN by GHA is available for forks. I'll try this, once #5088 is merged.

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.

6 participants