Skip to content

Make context macros self-contained #97

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
Feb 11, 2020
Merged

Make context macros self-contained #97

merged 1 commit into from
Feb 11, 2020

Conversation

bossmc
Copy link
Contributor

@bossmc bossmc commented Feb 11, 2020

With this, the various make_context_X macros never reference names from the local crate without fully qualifying them. Without this, users of the macros would have to use various swagger traits/macros (or #[macro_use] the crate).

@bossmc bossmc requested a review from richardwhiuk February 11, 2020 12:35
@richardwhiuk
Copy link
Contributor

bors delegate+

@bors
Copy link
Contributor

bors bot commented Feb 11, 2020

✌️ bossmc can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

Signed-off-by: Andy Caldwell <andrew.caldwell@metaswitch.com>
@bossmc
Copy link
Contributor Author

bossmc commented Feb 11, 2020

bors r+

bors bot added a commit that referenced this pull request Feb 11, 2020
97: Make context macros self-contained r=bossmc a=bossmc

With this, the various `make_context_X` macros never reference names from the local crate without fully qualifying them.  Without this, users of the macros would have to `use` various swagger traits/macros (or `#[macro_use]` the crate).

Co-authored-by: Andy Caldwell <andrew.caldwell@metaswitch.com>
@bors
Copy link
Contributor

bors bot commented Feb 11, 2020

Build succeeded

@bors bors bot merged commit 71b99eb into master Feb 11, 2020
@bors bors bot deleted the macro-ergonomics branch February 11, 2020 12:43
bors bot added a commit that referenced this pull request Feb 11, 2020
99: Re-instate accidentally removed test and fix follow-on issues r=bossmc a=bossmc

Whoops, accidentally commented out the doctest in #97.  Re-instate it and tighten it up to find a few straggling issues.

Co-authored-by: Andy Caldwell <andrew.caldwell@metaswitch.com>
bors bot added a commit that referenced this pull request Feb 11, 2020
99: Re-instate accidentally removed test and fix follow-on issues r=richardwhiuk a=bossmc

Whoops, accidentally commented out the doctest in #97.  Re-instate it and tighten it up to find a few straggling issues.

Co-authored-by: Andy Caldwell <andrew.caldwell@metaswitch.com>
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