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

Fixes #1662 - update whitelist for type-stylo label #1676

Merged
merged 1 commit into from
Jul 26, 2017

Conversation

cch5ng
Copy link
Contributor

@cch5ng cch5ng commented Jul 25, 2017

@miketaylr r?

do you think this visual usability bug matters?
#1662 (comment)

I verified that on submit, the issue gets saved with the appropriate github label so maybe it is not a big deal?

tests: webcompat#1662; add funct test; skip media report tests
@miketaylr
Copy link
Member

miketaylr commented Jul 26, 2017

@miketaylr r?

@cch5ng if you use the r? @miketaylr my mail filters catch this more quickly, sorry for the delay here. ^_^

edit: probably time to update my review filters...

@miketaylr
Copy link
Member

do you think this visual usability bug matters?

No, not really. That information is for the triagers and will be used at triage time, rather than report time. Let's land it.

@miketaylr
Copy link
Member

@cch5ng in the future, if you're making an unrelated change (the changes to tests), please include a separate commit for that. there's a number of good resources out there on "atomic commits", here's one that might be helpful: https://seesparkbox.com/foundry/atomic_commits_with_git

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

Successfully merging this pull request may close these issues.

3 participants