Skip to content

For CI - invert pipeline#3593

Closed
trianglesphere wants to merge 1 commit intodevelopfrom
jg/invert_pipeline_v2
Closed

For CI - invert pipeline#3593
trianglesphere wants to merge 1 commit intodevelopfrom
jg/invert_pipeline_v2

Conversation

@trianglesphere
Copy link
Contributor

@trianglesphere trianglesphere commented Sep 28, 2022

Description

A clear and concise description of the features you're adding in this pull request.

Tests

Please describe any tests you've added. If you've added no tests, or left important behavior untested, please explain why not.

Additional context

Add any other context about the problem you're solving.

Metadata

  • Fixes #[Link to Issue]

@changeset-bot
Copy link

changeset-bot bot commented Sep 28, 2022 •
edited
Loading

⚠️ No Changeset found

Latest commit: 3bd03fc

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mergify
Copy link
Contributor

mergify bot commented Sep 28, 2022

This PR changes implementation code, but doesn't include a changeset. Did you forget to add one?

@trianglesphere trianglesphere force-pushed the jg/invert_pipeline_v2 branch 3 times, most recently from be42c0d to 546d2f9 Compare September 28, 2022 22:49
@mergify
Copy link
Contributor

mergify bot commented Sep 28, 2022

Hey @trianglesphere! This PR has merge conflicts. Please fix them before continuing review.

@mergify mergify bot added the conflict label Sep 28, 2022
@trianglesphere
op-node: Clean up pipeline
3bd03fc 
Now that the engine queue is the only step stage, it is easy to
consolidate different loops inside the derivation pipeline.
@mergify mergify bot removed the conflict label Sep 28, 2022
@mergify mergify bot mentioned this pull request Sep 29, 2022
Closed
@mergify
Copy link
Contributor

mergify bot commented Sep 29, 2022

Hey @trianglesphere! This PR has merge conflicts. Please fix them before continuing review.

@mergify mergify bot added the conflict label Sep 29, 2022
@mergify mergify bot removed the conflict label Sep 30, 2022
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

@trianglesphere