Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Rust survey results (draft) #56

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

Merged
merged 4 commits into from
Feb 5, 2020
Merged

Rust survey results (draft) #56

merged 4 commits into from
Feb 5, 2020

Conversation

erlend-sh
Copy link
Member

A few changes still pending, in particular the wordcloud graphic for "libraries you're thankful for"

I'll share access to the form on Google Docs with the other WG leads.

A few changes still pending.
Copy link
Collaborator

@17cupsofcoffee 17cupsofcoffee left a comment

Choose a reason for hiding this comment

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

Looking really good so far 😄 Looks like we got some really useful feedback.

A few comments on the formatting:

The graphs look nice, but it'd be good if we could take a cue from the main Rust surveys and provide a little summary of the data alongside them? That way, if someone isn't able to view the image for whatever reason (e.g. screen readers) they'll not lose out.

Also, I really like the idea of displaying the 'signal strength' of each individual item, but I think it might be better to just show that as a number rather than a bar, so it's easier to compare.

@AlexEne
Copy link
Member

AlexEne commented Jan 29, 2020

This is really good data. Thanks for all the work you have done. So happy that map has debug visualization and cargo profiles will also solve the debug config pains :)

@erlend-sh erlend-sh marked this pull request as ready for review February 3, 2020 11:23
@erlend-sh
Copy link
Member Author

erlend-sh commented Feb 3, 2020

Alright, this is ready for final review now. I've incorporated your feedback @17cupsofcoffee , thanks!

Can be published at-will.

content/posts/survey-01/index.md Outdated Show resolved Hide resolved
[graphic pending]
Just for fun, we asked respondents to name libraries or projects they were thankful for. In the 6 months that have passed since this survey went out there's been quite a few newcomers on the rise that would probably show more prominently here if we asked again today. Rest assured there will be more surveys.
![Shoutouts to ecosystem projects](survey-thanks.png)
_Thanks so much to the survey respondents and all the people out there working tirelessly make Rust a first-tier language and ecosystem for game development._
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
_Thanks so much to the survey respondents and all the people out there working tirelessly make Rust a first-tier language and ecosystem for game development._
_Thank you so much to the survey respondents and all the people out there working tirelessly to make Rust a first-tier language and ecosystem for game development._

Copy link
Member Author

Choose a reason for hiding this comment

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

Has this been applied? I haven't used this GH feature before so I'm not quite sure what happened when I committed your first set of suggestions. These other suggestions are apparently "outdated" now.

Copy link
Contributor

Choose a reason for hiding this comment

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

The second suggestion seems not to have been applied.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, I guess I should’ve opted for “add suggestion to batch” instead. Could you please resubmit the missing suggestions?

Co-Authored-By: Wodann <Wodann@users.noreply.github.com>

### IDE/RLS story (28)

Tooling around IDE is limited compared to Visual Studio with C++ or C#.´
Copy link
Collaborator

Choose a reason for hiding this comment

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

There's a bullet point missing here and on lines 112/114.

I also kinda wonder if there's a way to format these bullets to make it a little clearer that they're quotes, but it's probably fine as is.

Copy link
Contributor

@Wodann Wodann Feb 3, 2020

Choose a reason for hiding this comment

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

I think marking it as quotes is actually a good point. The first time I read it, I wasn't sure whether they were a distilled, bullet-point summary of reflective quotes.

We can just use:

Quote goes here

Copy link
Collaborator

Choose a reason for hiding this comment

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

With the site's current styling, quotes would look like this:

image

If you do decide to go that route, please let me know if you're not happy with the fonts/italics/etc - I can submit a quick PR to change them before this post gets merged :)

@erlend-sh
Copy link
Member Author

Changed all to quotes, except for the last lists which were chiefly distillations rather than handpicked quotes. If that's deemed confusing that can still be changed.

I'll merge this now so we can start spreading the word shortly. Formatting changes after publishing won't be a problem; the data won't change after all.

@erlend-sh erlend-sh merged commit 64ffbe6 into rust-gamedev:source Feb 5, 2020
@17cupsofcoffee
Copy link
Collaborator

Looks good :) Thank you for all your hard work on this!

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.

5 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.