Skip to content
This repository was archived by the owner on Mar 4, 2026. It is now read-only.

Cleanup: Enforce Import ordering#498

Closed
schmidt-sebastian wants to merge 1 commit intomrschmidt-cleanupfrom
mrschmidt-imports
Closed

Cleanup: Enforce Import ordering#498
schmidt-sebastian wants to merge 1 commit intomrschmidt-cleanupfrom
mrschmidt-imports

Conversation

@schmidt-sebastian
Copy link
Copy Markdown
Contributor

@schmidt-sebastian schmidt-sebastian commented Dec 27, 2018

This is part of #512 but can be reviewed independently.

This PR cleans up all the imports for #510 and uses tslint to enforce ordering. I manually moved the imports around to group them into cohesive blocks before invoking the linter.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 27, 2018
@schmidt-sebastian schmidt-sebastian force-pushed the mrschmidt-imports branch 2 times, most recently from 39ae600 to fc1822b Compare December 28, 2018 00:03
@schmidt-sebastian schmidt-sebastian changed the base branch from master to mrschmidt-cleanup December 28, 2018 00:40
@JustinBeckwith JustinBeckwith added the 🚨 This issue needs some love. label Feb 7, 2019
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Feb 7, 2019
@JustinBeckwith JustinBeckwith added the needs work This is a pull request that needs a little love. label Feb 9, 2019
@yoshi-automation yoshi-automation removed the 🚨 This issue needs some love. label Feb 9, 2019
@schmidt-sebastian
Copy link
Copy Markdown
Contributor Author

Merged as part of #512

@schmidt-sebastian schmidt-sebastian deleted the mrschmidt-imports branch March 11, 2019 03:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement. needs work This is a pull request that needs a little love.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants