Skip to content

Add a test that define_encode_set works inside both modules and funct… #306

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

Merged
merged 1 commit into from
May 9, 2017

Conversation

brson
Copy link
Contributor

@brson brson commented May 4, 2017

Guidelines say to ensure item macros work in both positions, and I couldn't tell, so I added a test.


This change is Reviewable

@SimonSapin
Copy link
Member

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit dd5d84a has been approved by SimonSapin

@bors-servo
Copy link
Contributor

⌛ Testing commit dd5d84a with merge 356de6c...

bors-servo pushed a commit that referenced this pull request May 9, 2017
Add a test that define_encode_set works inside both modules and funct…

[Guidelines](https://github.com/brson/rust-api-guidelines#macros) say to ensure item macros work in both positions, and I couldn't tell, so I added a test.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/306)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - status-travis
Approved by: SimonSapin
Pushing 356de6c to master...

@bors-servo bors-servo merged commit dd5d84a into servo:master May 9, 2017
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.

3 participants