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

Refactor so package publishing only determines git root in one place #9851

Closed
wants to merge 2 commits into from

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