Skip to content

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

Merged
bors merged 1 commit into
rust-lang:masterfrom
eddyb:mac-name
Jan 26, 2020
Merged

Don't use ExpnKind::descr to get the name of a bang macro.#5096
bors merged 1 commit into
rust-lang:masterfrom
eddyb:mac-name

Conversation

@eddyb

@eddyb eddyb commented Jan 26, 2020

Copy link
Copy Markdown
Contributor

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

oli-obk commented Jan 26, 2020

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Jan 26, 2020

Copy link
Copy Markdown
Contributor

📌 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

bors commented Jan 26, 2020

Copy link
Copy Markdown
Contributor

⌛ Testing commit 0e52a0e with merge f690786...

@bors

bors commented Jan 26, 2020

Copy link
Copy Markdown
Contributor

💔 Test failed - checks-travis

@eddyb

eddyb commented Jan 26, 2020

Copy link
Copy Markdown
Contributor Author

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

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

@eddyb

eddyb commented Jan 26, 2020

Copy link
Copy Markdown
Contributor Author

@bors retry

@bors

bors commented Jan 26, 2020

Copy link
Copy Markdown
Contributor

@eddyb: 🔑 Insufficient privileges: not in try users

@llogiq

llogiq commented Jan 26, 2020

Copy link
Copy Markdown
Contributor

@bors retry

@bors

bors commented Jan 26, 2020

Copy link
Copy Markdown
Contributor

⌛ 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

bors commented Jan 26, 2020

Copy link
Copy Markdown
Contributor

☀️ 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

mati865 commented Jan 26, 2020

Copy link
Copy Markdown
Member

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
Copy Markdown
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

eddyb commented Jan 27, 2020

Copy link
Copy Markdown
Contributor Author

@flip1995 would moving to github actions fully fix this?

@flip1995

Copy link
Copy Markdown
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