Skip to content

Switching from public/private to external/internal APIs#36815

Merged
kobelb merged 2 commits intoelastic:masterfrom
kobelb:security-external-apis
May 22, 2019
Merged

Switching from public/private to external/internal APIs#36815
kobelb merged 2 commits intoelastic:masterfrom
kobelb:security-external-apis

Conversation

@kobelb
Copy link
Contributor

@kobelb kobelb commented May 21, 2019

We use the public folder a lot of other places to denote files which
should be exposed client-side. Using folders named public was causing
issues with some of the operations team's tooling, so we're renaming
these.

We use the public folder a lot of other places to denote files which
should be exposed client-side. Using folders names public was causing
issues with some of the operations team's tooling, so we're renaming
these.
@kobelb kobelb requested a review from legrego May 21, 2019 18:12
@kobelb kobelb requested a review from a team as a code owner May 21, 2019 18:12
@kobelb kobelb added the Team:Security Platform Security: Auth, Users, Roles, Spaces, Audit Logging, etc t// label May 21, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-security

@kobelb kobelb added chore release_note:skip Skip the PR/issue when compiling release notes labels May 21, 2019
Copy link
Contributor

@mistic mistic left a comment

Choose a reason for hiding this comment

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

LGTM ❤️

Copy link
Member

@legrego legrego left a comment

Choose a reason for hiding this comment

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

LGTM on green. Thanks for the quick turnaround @kobelb!

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@kobelb kobelb merged commit 18ef190 into elastic:master May 22, 2019
@kobelb kobelb deleted the security-external-apis branch May 22, 2019 13:41
kobelb added a commit to kobelb/kibana that referenced this pull request May 22, 2019
* Switching from public/private to external/internal APIs

We use the public folder a lot of other places to denote files which
should be exposed client-side. Using folders names public was causing
issues with some of the operations team's tooling, so we're renaming
these.

* Fixing forgotten import in test
kobelb added a commit that referenced this pull request May 24, 2019
)

* Switching from public/private to external/internal APIs

We use the public folder a lot of other places to denote files which
should be exposed client-side. Using folders names public was causing
issues with some of the operations team's tooling, so we're renaming
these.

* Fixing forgotten import in test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore release_note:skip Skip the PR/issue when compiling release notes Team:Security Platform Security: Auth, Users, Roles, Spaces, Audit Logging, etc t//

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants