Skip to content

[WIP] Update Angular styleguide.#9495

Closed
cjcenizal wants to merge 1 commit intomasterfrom
enh/new-angular-styleguide
Closed

[WIP] Update Angular styleguide.#9495
cjcenizal wants to merge 1 commit intomasterfrom
enh/new-angular-styleguide

Conversation

@cjcenizal
Copy link
Copy Markdown
Contributor

@cjcenizal cjcenizal commented Dec 15, 2016

Addresses #9049

Since this is a collaborative effort, let's make PRs into this branch and merge them (no review needed). Then we can do a final review of this PR alone.

References

new rules completed through to services, the rest is pulled directly from the john-papa styleguide
@cjcenizal cjcenizal requested review from spalger and w33ble December 15, 2016 00:23
@cjcenizal cjcenizal changed the title Update Angular styleguide. [WIP] Update Angular styleguide. Dec 15, 2016
@cjcenizal cjcenizal added Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// v5.2.0 v6.0.0 labels Dec 15, 2016
@spalger spalger self-assigned this Dec 16, 2016
@w33ble w33ble self-assigned this Dec 16, 2016
@epixa epixa added v5.2.1 and removed v5.2.0 labels Jan 31, 2017
@epixa epixa added v5.2.2 and removed v5.2.1 labels Feb 7, 2017
@cjcenizal cjcenizal added v5.4.0 and removed v5.2.2 labels Feb 9, 2017
@w33ble
Copy link
Copy Markdown
Contributor

w33ble commented Apr 10, 2017

@cjcenizal @spalger do we care about this anymore? With the intention to move to react, this seems pretty unimportant.

@cjcenizal
Copy link
Copy Markdown
Contributor Author

I agree. Closing.

@cjcenizal cjcenizal closed this Apr 10, 2017
@epixa epixa deleted the enh/new-angular-styleguide branch June 23, 2017 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// v5.4.0 v6.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants