Skip to content

Conversation

@Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Sep 16, 2021

This allows us to isolate our markdown rendering code in a dedicated package, which could potentially have a positive effect on the compile times of the full project. It also means we can iterate on this part of the code independent from the other parts (see #3861 or #3297), and with a well-defined API interface.

This allows us to isolate our markdown rendering code in a dedicated package, which could potentially have a positive effect on the compile times of the full project. It also means we can iterate on this part of the code independent from the other parts, and with a well-specified API interface.
@Turbo87 Turbo87 added C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear A-backend ⚙️ labels Sep 16, 2021
@jtgeibel
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Sep 17, 2021

📌 Commit f9871c2 has been approved by jtgeibel

@bors
Copy link
Contributor

bors commented Sep 17, 2021

⌛ Testing commit f9871c2 with merge 2df42ae...

@bors
Copy link
Contributor

bors commented Sep 17, 2021

☀️ Test successful - checks-actions
Approved by: jtgeibel
Pushing 2df42ae to master...

@bors bors merged commit 2df42ae into rust-lang:master Sep 17, 2021
@pietroalbini
Copy link
Member

Could we move it outside of src? It feels really confusing otherwise to have a subdirectory be a crate.

@Turbo87
Copy link
Member Author

Turbo87 commented Sep 17, 2021

@pietroalbini yeah, I think that makes sense. I've put it there because the cargo_registry_s3 is also inside of src, but I would definitely prefer it outside of it.

@Turbo87 Turbo87 deleted the cio-markdown branch September 17, 2021 21:29
bors added a commit that referenced this pull request Sep 26, 2021
Move subpackages to top-level folder

as requested in #3926 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants