Skip to content

ci: use buildifier binary release - #27489

Closed
alexeagle wants to merge 2 commits into
angular:masterfrom
alexeagle:skylint
Closed

alexeagle wants to merge 2 commits into
angular:masterfrom
alexeagle:skylint

Conversation

@alexeagle

Copy link
Copy Markdown
Contributor

Switch from Skylint to buildifier --lint - this is required for the Bazel 0.20 upgrade since Bazel no longer lets us use the embedded JDK to build and run Java programs, and Skylint is a Java program

@mary-poppins

Copy link
Copy Markdown

You can preview 4b04ed9 at https://pr27489-4b04ed9.ngbuilds.io/.

@alexeagle
alexeagle force-pushed the skylint branch 2 times, most recently from 3d3ba63 to e8427dc Compare December 5, 2018 20:42
@mary-poppins

Copy link
Copy Markdown

You can preview e8427dc at https://pr27489-e8427dc.ngbuilds.io/.

@mary-poppins

Copy link
Copy Markdown

You can preview 417e4f9 at https://pr27489-417e4f9.ngbuilds.io/.

@kyliau kyliau assigned kyliau and unassigned kyliau Dec 5, 2018
Switch from Skylint to buildifier --lint - this is required for the Bazel 0.20 upgrade since Bazel no longer lets us use the embedded JDK to build and run Java programs, and Skylint is a Java program
@mary-poppins

Copy link
Copy Markdown

You can preview f552501 at https://pr27489-f552501.ngbuilds.io/.

@alexeagle
alexeagle requested a review from IgorMinar December 6, 2018 22:58
@IgorMinar IgorMinar added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Dec 7, 2018
alxhub pushed a commit that referenced this pull request Dec 7, 2018
Switch from Skylint to buildifier --lint - this is required for the Bazel 0.20 upgrade since Bazel no longer lets us use the embedded JDK to build and run Java programs, and Skylint is a Java program

PR Close #27489
alxhub pushed a commit that referenced this pull request Dec 7, 2018
@alxhub alxhub closed this in cab5927 Dec 7, 2018
alxhub pushed a commit that referenced this pull request Dec 7, 2018
@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 14, 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 target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants