Skip to content

Conversation

@eps1lon
Copy link
Member

@eps1lon eps1lon commented Jan 13, 2021

Investigating #24303 (comment)

Next time Sync translations are committed we'll see what the actor is. Unclear why the approach works for me (github.actor != 'eps1lon') but not l10nbot

Using actor equality works: https://github.com/mui-org/material-ui/actions/runs/481976947
Using actor inequality with me works: https://github.com/mui-org/material-ui/actions/runs/481976947

Update: https://github.com/mui-org/material-ui/actions/runs/484135718 was skipped like intended

@mui-pr-bot
Copy link

mui-pr-bot commented Jan 13, 2021

No bundle size changes

Generated by 🚫 dangerJS against 11f32e3

@eps1lon eps1lon force-pushed the test/gh-actions-l10nbot branch from 6313d4a to a0b6ad6 Compare January 13, 2021 05:24
@eps1lon eps1lon changed the title [test] Fix expensive GH actions still runing for l10nbot [test] Debug expensive GH actions still runing for l10nbot Jan 13, 2021
@eps1lon eps1lon marked this pull request as ready for review January 13, 2021 06:05
@eps1lon eps1lon added the test label Jan 13, 2021
@eps1lon eps1lon merged commit d6e9733 into mui:next Jan 13, 2021
@eps1lon eps1lon deleted the test/gh-actions-l10nbot branch January 13, 2021 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants