Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix uefi-macros trybuild test#1183

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
phip1611 merged 1 commit intomainfrombishop-update-ui-test
May 28, 2024
Merged

Fix uefi-macros trybuild test#1183

phip1611 merged 1 commit intomainfrombishop-update-ui-test
May 28, 2024

Conversation

nicholasbishop
Copy link
Member

@nicholasbishopnicholasbishop commentedMay 28, 2024
edited
Loading

A difference in behavior between CI and local testing was observed for the entry_bad_return_type test. This is due to the way trybuild sets the--verbose flag, which is overwritten if RUSTFLAGS is set (as it always is for our CI jobs). Add code in xtask to ensure that--verbose still gets set correctly.

Also update entry_bad_return_type.stderr to match the expected current output.

Checklist

  • Sensible git history (for example, squash "typo" or "fix" commits). See theRewriting History guide for help.
  • Update the changelog (if necessary)

phip1611 reacted with rocket emoji
@nicholasbishopnicholasbishop marked this pull request as draftMay 28, 2024 01:13
@nicholasbishopnicholasbishopforce-pushed thebishop-update-ui-test branch 8 times, most recently from58cbaa4 to5333d54CompareMay 28, 2024 02:56
A difference in behavior between CI and local testing was observed for theentry_bad_return_type test. This is due to the way trybuild sets the `--verbose`flag, which is overwritten if RUSTFLAGS is set (as it always is for our CIjobs). Add code in xtask to ensure that `--verbose` still gets set correctly.Also update entry_bad_return_type.stderr to match the expected current output.
@nicholasbishopnicholasbishop changed the titlewip: Figuring out what's going on with the failing UI testFix uefi-macros trybuild testMay 28, 2024
@nicholasbishopnicholasbishop marked this pull request as ready for reviewMay 28, 2024 03:18
@phip1611
Copy link
Member

thanks for the fix!

@phip1611phip1611 added this pull request to themerge queueMay 28, 2024
Merged via the queue intomain with commit0119674May 28, 2024
@phip1611phip1611 deleted the bishop-update-ui-test branchMay 28, 2024 08:24
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@nicholasbishop@phip1611

[8]ページ先頭

©2009-2025 Movatter.jp