Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add --no-fail-fast to CI testing. #738

Merged
merged 1 commit into from
Nov 11, 2024
Merged

Add --no-fail-fast to CI testing. #738

merged 1 commit into from
Nov 11, 2024

Conversation

xStrom
Copy link
Member

@xStrom xStrom commented Nov 11, 2024

Most of the time is spent on compiling, so might as well see all the tests that fail.

Also the cargo test --doc step is moved after the screenshot failure upload to make it clearer that the doc tests don't produce any screenshots.

Copy link
Member

@DJMcNab DJMcNab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about the move of the doc tests, but otherwise looks good

@@ -269,6 +268,9 @@ jobs:
name: masonry-snapshot-tests-${{ matrix.os }}
path: masonry/src/widget/screenshots

- name: cargo test --doc
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm. We likely do want these to produce screenshots - see #501.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, it won't be a big task to move it if it makes sense with the new screenshot implementation. As for right now, it looks like we don't even have an implementation, so the details may change.

@xStrom xStrom added this pull request to the merge queue Nov 11, 2024
Merged via the queue into linebender:main with commit 73d33ee Nov 11, 2024
17 checks passed
@xStrom xStrom deleted the nextest branch November 11, 2024 14: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.

2 participants