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.

Sign up for GitHub

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

Jump to bottom

Combined tests and check #423

Merged
merged 1 commit into from
Sep 3, 2020
Merged

Combined tests and check #423

merged 1 commit into from
Sep 3, 2020

Conversation

AngelOnFira
Copy link
Contributor

This combines cargo check and cargo test, as is explained in more detail in #418. Please do not merge until I have an ok from @CleanCut 👌

@cart cart mentioned this pull request Sep 3, 2020
Closed
CleanCut
CleanCut approved these changes Sep 3, 2020
View reviewed changes
Copy link
Member

@CleanCut CleanCut left a comment

Choose a reason for hiding this comment

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

Comment is here: #418 (comment)

@cart cart merged commit a5f6cb0 into bevyengine:master Sep 3, 2020
@karroffel karroffel added A-Build-System Related to build systems or continuous integration C-Feature A new feature, making something new possible labels Sep 3, 2020
mrk-its pushed a commit to mrk-its/bevy that referenced this pull request Oct 6, 2020
@AngelOnFira @mrk-its
Combined tests and check (bevyengine#423)
e7deec8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CleanCut CleanCut CleanCut approved these changes

@cart cart cart approved these changes

Assignees
No one assigned
Labels
A-Build-System Related to build systems or continuous integration C-Feature A new feature, making something new possible
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AngelOnFira @cart @CleanCut @karroffel