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

Updates to tock-registers for publishing #1106

Merged
merged 3 commits into from Jul 14, 2018
Merged

Conversation

ppannuto
Copy link
Member

Pull Request Overview

This is blocked on #1105.

This finishes the remaining bits for publishing, specifically fixing up doctests and adding metadata to the manifest.

Testing Strategy

Compiling.

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make formatall.

@ppannuto ppannuto added blocked Waiting on something, like a different PR or a dependency. P-Upkeep This a relatively minor change, or one that is limited in scope, and requires less scrutiny. tock-libraries This affects libraries supported by the Tock project labels Jul 10, 2018
bradjc
bradjc previously requested changes Jul 10, 2018
Copy link
Contributor

@bradjc bradjc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here.

@ppannuto ppannuto dismissed bradjc’s stale review July 10, 2018 01:02

I don't think that review applies to any of the changes for this PR? It's already blocked on #1105.

@ppannuto ppannuto removed the blocked Waiting on something, like a different PR or a dependency. label Jul 14, 2018
The doc tests are not optimized for macros or top-level declration
examples, but here's the relevant bit for why this does what it does:
https://doc.rust-lang.org/beta/rustdoc/documentation-tests.html#documenting-macros
@ppannuto
Copy link
Member Author

Rebased. This should be good to go now.

@ppannuto ppannuto merged commit b4cd500 into master Jul 14, 2018
@ppannuto ppannuto deleted the publish-tock-registers branch July 14, 2018 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-Upkeep This a relatively minor change, or one that is limited in scope, and requires less scrutiny. tock-libraries This affects libraries supported by the Tock project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants