Skip to content

[infra] Remove the last path dependencies #88

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

Merged
merged 1 commit into from
Jul 11, 2023
Merged

Conversation

dcharkes
Copy link
Collaborator

@dcharkes dcharkes commented Jul 11, 2023

After this PR, all examples and integration test projects also build against published versions of the packages.

This means we do not run tests against path dependencies at all anymore. Testing this is tracked in #80.

@coveralls
Copy link

coveralls commented Jul 11, 2023

Coverage Status

coverage: 98.642%. remained the same
when pulling 45de509 on remove-path-deps
into 1e89ed9 on main.

@dcharkes dcharkes merged commit acad396 into main Jul 11, 2023
@dcharkes dcharkes deleted the remove-path-deps branch July 11, 2023 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants