X Tutup
Skip to content

tools(public_api): add a test verifying the public api of core#6309

Closed
vsavkin wants to merge 1 commit intoangular:masterfrom
vsavkin:public_api_spec
Closed

tools(public_api): add a test verifying the public api of core#6309
vsavkin wants to merge 1 commit intoangular:masterfrom
vsavkin:public_api_spec

Conversation

@vsavkin
Copy link
Copy Markdown
Contributor

@vsavkin vsavkin commented Jan 6, 2016

This PR adds a new gulp task check-public-api. The task checks the public API of main barrels using https://github.com/angular/ts-api-guardian.

@vsavkin vsavkin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jan 6, 2016
@vsavkin vsavkin force-pushed the public_api_spec branch 5 times, most recently from ae92f75 to 68f2140 Compare January 6, 2016 22:49
@jeffbcross jeffbcross assigned jeffbcross and unassigned IgorMinar Jan 25, 2016
@jeffbcross jeffbcross added pr_state: LGTM action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jan 25, 2016
@jeffbcross jeffbcross assigned rkirov and vsavkin and unassigned jeffbcross and rkirov Jan 25, 2016
@jeffbcross
Copy link
Copy Markdown
Contributor

@vsavkin looks good but needs rebase

@jeffbcross jeffbcross added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: merge The PR is ready for merge by the caretaker labels Jan 25, 2016
@vsavkin vsavkin added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Jan 26, 2016
@mary-poppins
Copy link
Copy Markdown

Merging PR #6309 on behalf of @rkirov to branch presubmit-rkirov-pr-6309.

@rkirov
Copy link
Copy Markdown
Contributor

rkirov commented Jan 27, 2016

looks like needs rebasing, js tests are failing
https://travis-ci.org/angular/angular/builds/105066377

@vsavkin
Copy link
Copy Markdown
Contributor Author

vsavkin commented Jan 27, 2016

@rkirov rebased

@mary-poppins
Copy link
Copy Markdown

Merging PR #6309 on behalf of @alexeagle to branch presubmit-alexeagle-pr-6309.

@mhevery mhevery closed this in 68a799a Jan 27, 2016
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

X Tutup