Skip to content

Report errors at macro definition, not expansion#29828

Merged
bors merged 3 commits into
rust-lang:masterfrom
sanxiyn:check-macro
Nov 16, 2015
Merged

Report errors at macro definition, not expansion#29828
bors merged 3 commits into
rust-lang:masterfrom
sanxiyn:check-macro

Conversation

@sanxiyn

@sanxiyn sanxiyn commented Nov 14, 2015

Copy link
Copy Markdown
Contributor

Fix #27409.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton alexcrichton added the relnotes Marks issues that should be documented in the release notes of the next release. label Nov 16, 2015
@alexcrichton

Copy link
Copy Markdown
Member

r? @nrc

@rust-highfive rust-highfive assigned nrc and unassigned alexcrichton Nov 16, 2015
@alexcrichton

Copy link
Copy Markdown
Member

also cc @sfackler

@nrc

nrc commented Nov 16, 2015

Copy link
Copy Markdown
Member

@bors: r+

@bors

bors commented Nov 16, 2015

Copy link
Copy Markdown
Collaborator

📌 Commit cce7b8b has been approved by nrc

@bors

bors commented Nov 16, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit cce7b8b with merge bf78389...

bors added a commit that referenced this pull request Nov 16, 2015
@bors bors merged commit cce7b8b into rust-lang:master Nov 16, 2015
@sanxiyn sanxiyn deleted the check-macro branch November 16, 2015 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

relnotes Marks issues that should be documented in the release notes of the next release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants