Skip to content

Conversation

nipunn1313
Copy link
Contributor

Refactor so package publishing only determines git root in one place

Depends on #9837

@rust-highfive
Copy link

r? @alexcrichton

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 28, 2021
@alexcrichton
Copy link
Member

@bors: r+

Thanks!

@bors
Copy link
Contributor

bors commented Aug 30, 2021

📌 Commit a9504cd3681e203aafdae16388689493793cd10d has been approved by alexcrichton

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 30, 2021
@alexcrichton
Copy link
Member

@bors: r-

carrying over #9837 (comment)

@bors bors added S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 30, 2021
@bors
Copy link
Contributor

bors commented Sep 2, 2021

☔ The latest upstream changes (presumably #9865) made this pull request unmergeable. Please resolve the merge conflicts.

Added unit tests for publishing with
- cargo.toml in root + readme in root
- cargo.toml in root + readme in subdir
- cargo.toml in subdir + readme next to it
- cargo.toml in subdir + readme in root (../README)

Should fix rust-lang/crates.io#3484 in
combination with rust-lang/crates.io#3861
@nipunn1313 nipunn1313 changed the title Readme path2 Refactor so package publishing only determines git root in one place Sep 2, 2021
@nipunn1313
Copy link
Contributor Author

(probably we should prefer something like #9866 to this)

@nipunn1313
Copy link
Contributor Author

Closing in favor of #9866

@nipunn1313 nipunn1313 closed this Sep 9, 2021
@nipunn1313 nipunn1313 deleted the readme_path2 branch September 26, 2021 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants