Skip to content

Developer Experience Audit #127

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
Merged
BRONSOLO merged 5 commits into from
May 3, 2017
Merged

Developer Experience Audit #127

BRONSOLO merged 5 commits into from
May 3, 2017

Conversation

@RichardLitt
Copy link
Contributor

@RichardLitt RichardLitt commented Apr 28, 2017

Hey there! These are documentation and licensing fixes:

  • I added a Code of Conduct.
  • I added an MIT license for Plotly, Inc.
  • I standardized the README, changed the extension ending to .md, added a contribute and license section, and renamed some of the headings. It should be more readable now.
@BRONSOLO
Copy link
Member

BRONSOLO commented May 2, 2017

This Looks great @RichardLitt! We would like to keep the README as a .mkdn to force a 404 for old wrappers trying to upgrade. Could you please change that back. (For reference: #125 (comment)).
@RichardLitt
Copy link
Contributor Author

RichardLitt commented May 3, 2017

Hey @BRONSOLO! Sweet. Done.
@chriddyp
Copy link
Member

chriddyp commented May 3, 2017

Looks great to me! 💃
@BRONSOLO BRONSOLO merged commit 998df11 into plotly:master May 3, 2017
@BRONSOLO
Copy link
Member

BRONSOLO commented May 3, 2017

Thanks again @RichardLitt!
@RichardLitt RichardLitt deleted the feat/dx-audit branch May 4, 2017 07:24
@RichardLitt
Copy link
Contributor Author

RichardLitt commented May 4, 2017

Awesome! \o/
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@RichardLitt @BRONSOLO @chriddyp