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.

Sign up for GitHub

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

Jump to bottom

Orgs using D: AdRoll #1343

Merged
merged 1 commit into from
Jun 6, 2016
Merged

Orgs using D: AdRoll #1343

merged 1 commit into from
Jun 6, 2016

Conversation

wilzbach
Copy link
Member

@wilzbach wilzbach commented Jun 6, 2016

And I just got the go-ahead.

@CyberShadow
Copy link
Member

I know this is silly, but can you rename adroll.png to something else so it bypasses ad blockers?

@wilzbach
Copy link
Member Author

wilzbach commented Jun 6, 2016

to something else so it bypasses ad blockers?

Ideas? roll_ad.png?

@JackStouffer
Copy link
Member

Best not to have ad in the name

@wilzbach
Copy link
Member Author

wilzbach commented Jun 6, 2016

Best not to have ad in the name

Okay I renamed it to da_roll.png. Does that bypass your adblock @CyberShadow?

@CyberShadow
Copy link
Member

Yep

@CyberShadow
Copy link
Member

"Integral parts of Adroll's business on D. " Where's the verb?

appascoe
appascoe reviewed Jun 6, 2016
View reviewed changes
orgs-using-d.dd
$(DORG Adroll, https://www.adroll.com, adroll.png,
Retargeting platform,
$(DORG AdRoll, https://www.adroll.com, da_roll.png,
Marketing Platform,
$(HTTP tech.adroll.com/blog/data-science/2015/12/08/data-science_event_processing.html, Integral parts) of $(HTTP tech.adroll.com/blog/data-science/2015/08/25/factorization-machines.html,Adroll's business) on D.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need a capital R here in AdRoll's business.

@appascoe
Copy link

appascoe commented Jun 6, 2016

Don't think it needs a verb, as it's just indicating links that describe parts of the business.

@wilzbach
Copy link
Member Author

wilzbach commented Jun 6, 2016

Need a capital R here in AdRoll's business.

Oh damn - sorry!

Don't think it needs a verb, as it's just indicating links that describe parts of the business.

I added "are built", does this make it clearer?

Disclaimer: I am not a native speaker:

@CyberShadow CyberShadow merged commit af69c15 into dlang:master Jun 6, 2016
@wilzbach wilzbach deleted the adroll branch June 8, 2016 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@wilzbach @CyberShadow @JackStouffer @appascoe