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

Add icon to image picker warning #3075

Merged
merged 1 commit into from
Feb 1, 2016

Conversation

anitagraham
Copy link
Contributor

Also change warning to clearer, more succinct message

@anitagraham
Copy link
Contributor Author

  1. don't take shortcuts
  2. always run tests locally

Some people live a life comptemplative or a life more ordinary. I am living a life more interrupted.

@bricesanchez
Copy link
Member

@anitagraham Could you squash your commits ?

@bricesanchez bricesanchez self-assigned this Nov 30, 2015
@bricesanchez bricesanchez added this to the 3.1 milestone Nov 30, 2015
@anitagraham
Copy link
Contributor Author

Will squash. when the moment arises.

Also change warning to clearer, more succinct  message
@parndt
Copy link
Member

parndt commented Dec 7, 2015

Seems to fail CI (still)

@parndt
Copy link
Member

parndt commented Dec 7, 2015

Seems like #3069 is failing for the same problem, though? Maybe there was a change to another library we use?

bricesanchez added a commit that referenced this pull request Feb 1, 2016
@bricesanchez bricesanchez merged commit 6d5ac0a into refinery:master Feb 1, 2016
@bricesanchez
Copy link
Member

Thanks @anitagraham !

@bricesanchez bricesanchez removed this from the 3.1 milestone Apr 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants