Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[derive] Don't emit #[cfg(coverage_nightly)] (#2123) #2126

Merged
merged 1 commit into from
Dec 3, 2024
Merged

Conversation

joshlf
Copy link
Member

@joshlf joshlf commented Dec 3, 2024

As of nightly-2024-11-20 - specifically [1] - this triggers an unexpected_cfgs lint even when emitted in derive-generated code.

[1] rust-lang/rust#132577

Fixes #2117

As of nightly-2024-11-20 - specifically [1] - this triggers an
`unexpected_cfgs` lint even when emitted in derive-generated code.

[1] rust-lang/rust#132577

Fixes #2117

gherrit-pr-id: I9d536f0ea08475afe0b65411b225aa55f4db449a
@joshlf joshlf requested a review from jswrenn December 3, 2024 15:54
@joshlf joshlf enabled auto-merge December 3, 2024 15:54
@joshlf
Copy link
Member Author

joshlf commented Dec 3, 2024

Backport of #2123

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.93%. Comparing base (775a1ba) to head (7e91ebe).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2126   +/-   ##
=======================================
  Coverage   89.93%   89.93%           
=======================================
  Files          16       16           
  Lines        5833     5833           
=======================================
  Hits         5246     5246           
  Misses        587      587           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@joshlf joshlf added this pull request to the merge queue Dec 3, 2024
Merged via the queue into main with commit 9a2fce6 Dec 3, 2024
69 checks passed
@joshlf joshlf deleted the backport-fix-2117 branch December 3, 2024 16:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The KnownLayout macro has an issue with the latest nightly
3 participants