Skip to content

Conversation

scrye
Copy link
Contributor

@scrye scrye commented Jan 10, 2020

This gives us a quick idea of what still needs to be ported.


This change is Reviewable

@scrye scrye added the i/devops Everything related to CI/Bazel/building label Jan 10, 2020
@scrye scrye requested a review from lukedirtwalker January 10, 2020 09:20
@scrye scrye self-assigned this Jan 10, 2020
Copy link
Collaborator

@lukedirtwalker lukedirtwalker left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @scrye)

a discussion (no related file):
Remove the icon from lint, since it is not completely "bazelified"


Copy link
Contributor Author

@scrye scrye left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 of 2 files reviewed, 1 unresolved discussion (waiting on @lukedirtwalker and @scrye)

a discussion (no related file):

Previously, lukedirtwalker (Lukas Vogel) wrote…

Remove the icon from lint, since it is not completely "bazelified"

Done.


scrye added 2 commits January 10, 2020 10:30
This gives us a quick idea of what still needs to be ported.
Copy link
Collaborator

@lukedirtwalker lukedirtwalker left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r2.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@scrye scrye merged commit 715c1f1 into scionproto:master Jan 10, 2020
@scrye scrye deleted the pubpr-hint-bazel branch January 10, 2020 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i/devops Everything related to CI/Bazel/building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants