Skip to content

fix: remove doc_auto_cfg on docsrs#310

Merged
ayrat555 merged 1 commit intoayrat555:masterfrom
BlindMaster24:fix/docs-build
Jan 19, 2026
Merged

fix: remove doc_auto_cfg on docsrs#310
ayrat555 merged 1 commit intoayrat555:masterfrom
BlindMaster24:fix/docs-build

Conversation

@BlindMaster24
Copy link
Contributor

No description provided.

@EdJoPaTo
Copy link
Collaborator

You should explain why you think this is helpful. Please take some effort in creating Pull Requests if you want them to get merged. Personally I think this is useful and should not be removed.

@BlindMaster24
Copy link
Contributor Author

@BlindMaster24
Copy link
Contributor Author

[INFO] [stderr] 1 | #![cfg_attr(docsrs, feature(doc_auto_cfg))]
[INFO] [stderr] | ^^^^^^^^^^^^ feature has been removed
[INFO] [stderr] |
[INFO] [stderr] = note: removed in 1.92.0; see rust-lang/rust#138907 for more information
[INFO] [stderr] = note: merged into doc_cfg
[INFO] [stderr]
[INFO] [stderr] error: Compilation failed, aborting rustdoc

@ayrat555
Copy link
Owner

thank you @BlindMaster24 for fixing this
thank you @EdJoPaTo for staying vigilant

@ayrat555 ayrat555 merged commit e3dcf4e into ayrat555:master Jan 19, 2026
32 checks passed
@ayrat555
Copy link
Owner

@BlindMaster24 I'm sorry I had remove your commit. it seems it's not completely correct

@EdJoPaTo
Copy link
Collaborator

@BlindMaster24 Being snarky is not helpful when trying to merge something in some repo. Being wrong with the change does not help either.

Read what you posted there. Multiple features were merged into a single one. Removing it results in not having the wanted feature. Using the feature it got merged into both compiles and shows the required features of certain objects as its wanted. Fixed that issue with #312

@BlindMaster24 BlindMaster24 deleted the fix/docs-build branch February 6, 2026 03:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants