Skip to content

Conversation

workingjubilee
Copy link
Member

Some platforms were not linked from platform-support.md

This fixes that, but errs towards extremely conservative, only directly linking platform docs if the docs actively mention the target, as otherwise I do not necessarily know if there was a reason for the omission.

@rustbot
Copy link
Collaborator

rustbot commented Jul 16, 2023

r? @GuillaumeGomez

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 16, 2023
@workingjubilee
Copy link
Member Author

workingjubilee commented Jul 16, 2023

For examples of targets that I noticed did have platform support docs that might be relevant, but weren't directly linked:

  • powerpc-unknown-openbsd vs. unmentioned in openbsd.md
  • riscv64gc-unknown-fuchsia vs. unmentioned in fuchsia.md

RV64GC isn't supported by the Fuchsia SDK, as I understand it, so that's why it isn't documented clearly. And they're both tier 3 anyways so currently my feeling is "whatever". This was spurred by #113739

@GuillaumeGomez
Copy link
Member

r? @joshtriplett

Some platforms were not linked from platform-support.md

This fixes that, but errs towards extremely conservative,
only directly linking platform docs if the docs actively
mention the target, as otherwise I do not necessarily
know if there was a reason for the omission.
@workingjubilee workingjubilee force-pushed the link-more-platform-support-docs branch from 244d0b0 to 467cb52 Compare August 1, 2023 03:12
@joshtriplett
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Aug 1, 2023

📌 Commit 467cb52 has been approved by joshtriplett

It is now in the queue for this repository.

@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 1, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 1, 2023
…iaskrgr

Rollup of 7 pull requests

Successful merges:

 - rust-lang#111081 (impl SliceIndex<str> for (Bound<usize>, Bound<usize>))
 - rust-lang#113394 (style-guide: Document style editions, start 2024 style edition)
 - rust-lang#113588 (bootstrap: use git merge-base for LLVM CI download logic)
 - rust-lang#113743 (Directly link more target docs)
 - rust-lang#114262 (Improve the rust style guide doc)
 - rust-lang#114309 (Update books)
 - rust-lang#114313 ([rustc_data_structures] Simplify SortedMap::insert.)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 4e60d99 into rust-lang:master Aug 1, 2023
@rustbot rustbot added this to the 1.73.0 milestone Aug 1, 2023
@workingjubilee workingjubilee deleted the link-more-platform-support-docs branch June 6, 2025 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants