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

orgs using d - update weka's description #1334

Merged
merged 1 commit into from
Jun 6, 2016

Conversation

wilzbach
Copy link
Member

@wilzbach wilzbach commented Jun 5, 2016

Follow-up to #1312

CC @liranz

Is the size of Weka's codebase something that should be mentioned or should I better remove that?

@liranz
Copy link

liranz commented Jun 5, 2016

You can use our current code base size. It won't stop us from writing new code :)
The impressive part of the figure is that comparable projects are much larger, so it's a tribute to D's "efficiency" as a programming language.

@wilzbach
Copy link
Member Author

wilzbach commented Jun 5, 2016

You can use our current code base size. It won't stop us from writing new code :)
The impressive part of the figure is that comparable projects are much larger, so it's a tribute to D's "efficiency" as a programming language.

Good point - should I change it to "only"?

@JackStouffer
Copy link
Member

Good point - should I change it to "only"?

If you do so, be sure to include the sizes of comparable projects as a reference, as 200K sounds huge at first glance.

@wilzbach
Copy link
Member Author

wilzbach commented Jun 5, 2016

If you do so, be sure to include the sizes of comparable projects as a reference, as 200K sounds huge at first glance.

Hmm I am limited to only two lines, so I wouldn't know how. Do you have an idea?

@JackStouffer
Copy link
Member

I'd just leave the part about "only" out then.

@CyberShadow CyberShadow merged commit b31edb5 into dlang:master Jun 6, 2016
@wilzbach wilzbach deleted the update-weka branch June 8, 2016 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants