Skip to content
This repository was archived by the owner on Jan 24, 2022. It is now read-only.

v0.6.0, take 2 #104

Merged
merged 1 commit into from
Sep 6, 2018
Merged

v0.6.0, take 2 #104

merged 1 commit into from
Sep 6, 2018

Conversation

japaric
Copy link
Member

@japaric japaric commented Sep 6, 2018

the dev-dependency of cortex-m-rt-macros needs to specify a version or can't be
published; this commit fixes that

docs.rs won't be able to build docs for these two crates so this also changes
the documentation link to https://rust-embedded.github.io/cortex-m-rt/ where I
have (manually) implemented a poor man's docs.rs

now we should be able to publish a new version

r? @rust-embedded/cortex-m (anyone)

the dev-dependency of cortex-m-rt-macros needs to specify a version or can't be
published; this commit fixes that

docs.rs won't be able to build docs for these two crates so this also changes
the documentation link to https://rust-embedded.github.io/cortex-m-rt/ where I
have (manually) implemented a poor man's docs.rs
@japaric japaric requested a review from a team as a code owner September 6, 2018 14:57
Copy link
Contributor

@therealprof therealprof left a comment

Choose a reason for hiding this comment

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

LGTM

@therealprof
Copy link
Contributor

bors r+

bors bot added a commit that referenced this pull request Sep 6, 2018
104: v0.6.0, take 2 r=therealprof a=japaric

the dev-dependency of cortex-m-rt-macros needs to specify a version or can't be
published; this commit fixes that

docs.rs won't be able to build docs for these two crates so this also changes
the documentation link to https://rust-embedded.github.io/cortex-m-rt/ where I
have (manually) implemented a poor man's docs.rs

now we should be able to publish a new version

r? @rust-embedded/cortex-m (anyone)

Co-authored-by: Jorge Aparicio <[email protected]>
@bors
Copy link
Contributor

bors bot commented Sep 6, 2018

Build succeeded

@bors bors bot merged commit aabae79 into master Sep 6, 2018
@bors bors bot deleted the release branch September 6, 2018 15:08
@japaric
Copy link
Member Author

japaric commented Sep 6, 2018

This is awesome. We can't publish cortex-m-rt v0.6.0 because it depends on cortex-m-rt-macros v0.1.0, but we can't publish cortex-m-rt-macros v0.1.0 either because it depends on cortex-m-rt v0.6.0. 😂

I'll cheat a bit.

@japaric
Copy link
Member Author

japaric commented Sep 6, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants