-
Notifications
You must be signed in to change notification settings - Fork 21
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
Replace string ":" with "." on filename of screenshotName #49
base: master
Are you sure you want to change the base?
Conversation
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 👍
Would be nice to add a test for this change though.
Added test case |
@felangel what no merge? I have more actions in this fix? |
@jorgecoca are you able to merge? I don’t have permission. |
@dakrawczyk can you take a look? |
@JenniferJesuraj we may need another review on this PR before it's mergeable. |
@dakrawczyk looks like the ci isn't running. Do you have access to take a look? FYI @jorgecoca |
Hey @felangel ! yeah, I'm communicating with the team to get this resolved. Seems to be happening on our other repos as well. |
This fix solve the issue 48 https://github.com/bmw-tech/ozzie.flutter/issues/48