Skip to content
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

Meta: update repository files #51

Merged
merged 1 commit into from
Jun 15, 2020

Conversation

annevk
Copy link
Member

@annevk annevk commented Apr 23, 2020

@foolip foolip force-pushed the meta-template/ffdd9f7c-8571-11ea-b445-82d5a4c81c00 branch from 46e293b to 3f94207 Compare June 13, 2020 19:36
@foolip
Copy link
Member

foolip commented Jun 13, 2020

I've updated this with the latest stuff for whatwg/meta#173.

@foolip foolip mentioned this pull request Jun 13, 2020
25 tasks
@@ -4,13 +4,16 @@ root = true
end_of_line = lf
insert_final_newline = true
charset = utf-8
indent_size = 1
indent_size = 2
Copy link
Member

Choose a reason for hiding this comment

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

This change should be fine because *.bs gets indent 1 below.

@annevk
Copy link
Member Author

annevk commented Jun 14, 2020

Filed w3c/csswg-drafts#5209 on the build issue.

I wonder if we can surface the build failures in a better way as with GitHub Actions there's one more click to get to them.

@foolip
Copy link
Member

foolip commented Jun 14, 2020

I wonder if we can surface the build failures in a better way as with GitHub Actions there's one more click to get to them.

For anything associated with a line number we can do something pretty sweet, but we'd need speced/bikeshed#1686 fixed for it to be reliable.

@annevk
Copy link
Member Author

annevk commented Jun 14, 2020

So per https://help.github.com/en/actions/reference/workflow-commands-for-github-actions#setting-an-error-message line/col are optional. If we omit those, will it still surface inline somehow?

@foolip
Copy link
Member

foolip commented Jun 14, 2020

IIRC from my experiment, annotations for files that aren't touched would show up in the annotations list, much like the annotation for failure on this very PR:
https://github.com/whatwg/quirks/actions/runs/134485394

However, you don't see those unless you go looking.

One could pretend that warnings/errors are on the first line of a file, of course.

@annevk
Copy link
Member Author

annevk commented Jun 15, 2020

I see. I was hoping for information in the PR itself and failing that, information once you navigate to https://github.com/whatwg/quirks/pull/51/checks. But it seems you have to navigate there, then expand the failing step, and then maybe scroll, depending.

@annevk annevk requested a review from foolip June 15, 2020 12:10
@foolip foolip merged commit 89f4fe2 into master Jun 15, 2020
@foolip foolip deleted the meta-template/ffdd9f7c-8571-11ea-b445-82d5a4c81c00 branch June 15, 2020 13:05
@foolip
Copy link
Member

foolip commented Jun 15, 2020

It worked: https://quirks.spec.whatwg.org/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants