-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix exported comments enforcer in CI #5180
Conversation
f0d7f29
to
1d86854
Compare
Add comments where missing and fix incorrect comments Signed-off-by: Derek McGowan <derek@mcg.dev>
Signed-off-by: Derek McGowan <derek@mcg.dev>
1d86854
to
8cf669c
Compare
Build succeeded.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice, LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
very nice
sigh |
@thaJeztah actually seems like the macos instances don't dump any identifier; seems like only the Linux hosts, so we have nothing to go on: https://pipelines.actions.githubusercontent.com/G4SighzWVVZ6vsyiz7FFMFjLjRzveJHseEnVyibkSq87Cl2x4O/_apis/pipelines/1/runs/4118/signedlogcontent/64?urlExpires=2021-03-12T17%3A42%3A14.7222002Z&urlSigningMethod=HMACV1&urlSignature=HSjFqaYg9yGT0FHgi1L0hvT1BjNqY%2BiDzBQUmGqWKLk%3D |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I hate it that they disable it by default
Arf.. that's a pity 😞 |
Add comments where missing and fix incorrect comments