Skip to content

some cleanup extracted from the rebase flow changes#7129

Merged
shiftkey merged 9 commits intodevelopmentfrom
shiftkey/rebase/pre-tech-debt
Mar 25, 2019
Merged

some cleanup extracted from the rebase flow changes#7129
shiftkey merged 9 commits intodevelopmentfrom
shiftkey/rebase/pre-tech-debt

Conversation

@shiftkey
Copy link
Member

Overview

This is a pre-requisite for the work in #7117, which can land in advance of that review.

Description

I'm going to annotate the PR with context about these changes, but the commits are atomic enough that hopefully the rest makes sense.

Release notes

Notes: no-notes

@shiftkey shiftkey force-pushed the shiftkey/rebase/pre-tech-debt branch 2 times, most recently from 3ff1aeb to 4e750e8 Compare March 20, 2019 18:57
@shiftkey shiftkey added tech-debt Issues and pull requests related to addressing technical debt or improving the codebase ready-for-review Pull Requests that are ready to be reviewed by the maintainers labels Mar 20, 2019
@shiftkey shiftkey marked this pull request as ready for review March 20, 2019 18:59
@shiftkey shiftkey added this to the 1.7.0 milestone Mar 20, 2019
@shiftkey shiftkey force-pushed the shiftkey/rebase/pre-tech-debt branch from e33cfc1 to a1beb21 Compare March 21, 2019 14:56
@shiftkey shiftkey force-pushed the shiftkey/rebase/pre-tech-debt branch from a1beb21 to 1f00d9c Compare March 25, 2019 12:59
Copy link
Contributor

@outofambit outofambit left a comment

Choose a reason for hiding this comment

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

your inline comments were super helpful thank you! left some thoughts all over the place :) but nothing major

@outofambit outofambit self-assigned this Mar 25, 2019
@shiftkey shiftkey requested a review from outofambit March 25, 2019 17:21
Copy link
Contributor

@outofambit outofambit left a comment

Choose a reason for hiding this comment

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

✨ thank you!

@shiftkey shiftkey merged commit 7e58a4f into development Mar 25, 2019
@shiftkey shiftkey deleted the shiftkey/rebase/pre-tech-debt branch March 26, 2019 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-review Pull Requests that are ready to be reviewed by the maintainers tech-debt Issues and pull requests related to addressing technical debt or improving the codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants