-
Notifications
You must be signed in to change notification settings - Fork 259
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
Image attribute #565
Image attribute #565
Conversation
024568b
to
bcaa57a
Compare
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.
Thanks for jumping in here! This is looking super close. I've left a few comments in-line.
with the latest fixups, acceptance tests are passing
|
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.
👍 Looks great!
Hey, @rienafairefr - Thanks a ton for this sweet PR! 😄 Would you please shoot me an email when you get a chance? |
* image attribute * try adding acceptance test * fixup! image attribute * Update digitalocean/resource_digitalocean_app_test.go Co-authored-by: Andrew Starr-Bochicchio <[email protected]> * test acc image in its own function * move 'image' attribute to service/worker, exclude from static_site * fixup acceptance test * Remove commented out line. Co-authored-by: Andrew Craven <[email protected]> Co-authored-by: Matthieu Berthomé <[email protected]> Co-authored-by: Andrew Starr-Bochicchio <[email protected]> Co-authored-by: Andrew Starr-Bochicchio <[email protected]>
I extracted the
image
attribute thing from #544, and added acceptance tests which are still running so not sure they pass, this might still not be fully ready, but hey, figure I'd try.