8000 Run tests from source instead of building first by markerikson · Pull Request #4483 · reduxjs/redux · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
/ redux Public

Run tests from source instead of building first #4483

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
Merged
merged 2 commits into from
Feb 12, 2023
Merged

Run tests from source instead of building first #4483

merged 2 commits into from
Feb 12, 2023

Conversation

markerikson
Copy link
Contributor
@markerikson markerikson commented Feb 12, 2023 •
edited
Loading

Our test command has always done a full lint+build step every time, which is annoying.

I've reworked it so that test runs against src.

Also, apparently we don't actually do typetests against the usual range of TS versions like we do in all of our other repos. Added that.

@codesandbox-ci
Copy link
codesandbox-ci bot commented Feb 12, 2023 •
edited
Loading

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit bad6237:

Sandbox Source
Vanilla Configuration
Vanilla Typescript Configuration

8000
@markerikson markerikson force-pushed the feature/local-tests branch from 7768685 to 60ef15e Compare February 12, 2023 19:33
@markerikson markerikson merged commit 29d5d88 into master Feb 12, 2023
@markerikson markerikson deleted the feature/local-tests branch February 12, 2023 19:45
This was referenced Aug 9, 2024
Open
Open
Open
Open
@lucentlabz lucentlabz mentioned this pull request Sep 10, 2024
Merged
Sign up for free to 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.
1 participant
@markerikson
0