-
Notifications
You must be signed in to change notification settings - Fork 370
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #753 from JuliaStats/news
Updating doc and cleaning house for overdue release
- Loading branch information
Showing
19 changed files
with
213 additions
and
398 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,29 @@ | ||
# Style Guidelines | ||
Thanks for taking the plunge! | ||
|
||
* Include spaces after commmas | ||
## Reporting Issues | ||
|
||
* It's always good to start with a quick search for an existing issue to post on, or related issues for context, before opening a new issue | ||
* Including minimal examples is greatly appeciated | ||
* If it's a bug, or unexpected behaviour, reproducing on the latest development version (`Pkg.checkout("DataFrames")`) is a good gut check and can streamline the process, along with including the first two lines of output from `versioninfo()` | ||
|
||
## Contributing | ||
|
||
* Feel free to open, or comment on, an issue and solicit feedback early on, especially if you're unsure about aligning with design goals and direction, or if relevant historical comments are ambiguous | ||
* Pair new functionality with tests, and bug fixes with tests that fail pre-fix. Increasing test coverage as you go is always nice | ||
* Aim for atomic commits, if possible, e.g. `change 'foo' behavior like so` & `'bar' handles such and such corner case`, rather than `update 'foo' and 'bar'` & `fix typo` & `fix 'bar' better` | ||
* Pull requests are tested against release and development branches of Julia, so using `Pkg.test("DataFrames")` as you develop can be helpful | ||
* The style guidelines outlined below are not the personal style of most contributors, but for consistency throughout the project, we've adopted them | ||
|
||
## Style Guidelines | ||
|
||
* Include spaces | ||
+ After commas | ||
+ Around operators: `=`, `<:`, comparison operators, and generally around others | ||
+ But not after opening parentheses or before closing parentheses | ||
* Use four spaces for indentation (test data files and Makefiles excepted) | ||
* Don't leave trailing whitespace at the end of lines | ||
* Don't go over the 79 per-line character limit | ||
* Avoid squashing code blocks onto one line, e.g. `for foo in bar; baz += qux(foo); end` | ||
* Don't explicitly parameterize types unless it's necessary | ||
* Never leave things without type qualifications. Use an explicit Any. | ||
* Never leave things without type qualifications. Use an explicit `::Any`. | ||
* Order method definitions from most specific to least specific type constraints |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,15 @@ | ||
DataFrames.jl | ||
============= | ||
|
||
[![Build Status](https://travis-ci.org/JuliaStats/DataFrames.jl.svg?branch=master)](https://travis-ci.org/JuliaStats/DataFrames.jl) | ||
[![Coverage Status](http://img.shields.io/coveralls/JuliaStats/DataFrames.jl.svg)](https://coveralls.io/r/JuliaStats/DataFrames.jl) | ||
[![DataFrames](http://pkg.julialang.org/badges/DataFrames_release.svg)](http://pkg.julialang.org/?pkg=DataFrames&ver=release) | ||
[![Coverage Status](http://img.shields.io/coveralls/JuliaStats/DataFrames.jl.svg)](https://coveralls.io/r/JuliaStats/DataFrames.jl) | ||
[![Build Status](https://travis-ci.org/JuliaStats/DataFrames.jl.svg?branch=master)](https://travis-ci.org/JuliaStats/DataFrames.jl) | ||
[![Build status](https://ci.appveyor.com/api/projects/status/github/JuliaStats/DataFrames.jl?svg=true&branch=master)](https://ci.appveyor.com/project/garborg/dataframes-jl/branch/master) | ||
|
||
Tools for working with tabular data in Julia. | ||
|
||
Tools for working with data in Julia. | ||
**Installation**: at the Julia REPL, `Pkg.add("DataFrames")` | ||
|
||
**Installation**: at the Julia REPL: `Pkg.add("DataFrames")` | ||
**Documentation**: [available here](http://dataframesjl.readthedocs.org/en/latest/) | ||
|
||
**Documentation**: [available here](http://dataframesjl.readthedocs.org/en/latest/). | ||
**Reporting Issues and Contributing**: See [CONTRIBUTING.md](CONTRIBUTING.md) |
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.