Skip to content

Latest commit

 

History

History
110 lines (67 loc) · 5.26 KB

contributing.md

File metadata and controls

110 lines (67 loc) · 5.26 KB
Raw

#Contributing

Contributing back to Elasticsearch.Net and NEST is very much appreciated. Whether you feel the need to change one character or have a go at mapping new API's no PR is too small or too big.

In fact many of our most aweseome features/fixes have been provided to us by these wonderful folks to which we are forever indebted

Sign the CLA

We do ask that you sign the Elasticsearch CLA before we can accept pull requests from you.

Coding Styleguide

Please install the editorconfig vs extension this will automatically switch to our indentation, whitespace, newlines settings while working on our project while leaving your default settings intact.

In most cases we won't shun a PR just because it uses the wrong indentation settings though it'll be very much appreciated!

Tests

PR's with tests are more likely to be reviewed faster because it makes our jobs that much more easy, that being said we respect that you are fixing a bug in your own time and might not have the time/energy to submit a PR with complete tests. In those cases we tend to pull your bits locally and write tests ourselves but this may mean your PR might sit idle longer then you would like.

Branches

  • 1.x for latest 1.x compatible client
  • 2.x for 2.x compatible client
  • master for the latest client (currently 5.x alpha)

Git

We do not require rebased/squashed commits although we do very much appreciate it!

Please submit your Pull Requests to

  • master branch for 5.x
  • 2.x branch for 2.x
  • 1.x branch for 1.x

#Building the solution

The solution uses a number of awesome Open Source software tools to ease development:

##Paket

Paket is the dependency manager of choice for handling dependencies of both the solution and the build automation system. It works for both .NET and Mono, with an ability to reference packages from Nuget and also files directly from github.

##FAKE

FAKE (F# MAKE) is used as the build automation system for the solution. To get started after cloning the solution, it's best to run the build script in the root

for Windows

build.bat

for OSX/Linux

build.sh

This will

  • Pull down all the paket dependencies for the build process as well as the solution
  • Run the default build target for the solution

You can also compile the solution within Visual Studio if you prefer, but the build script is going to be faster.

##Tests

The Tests project contains both xunit unit and integration tests. A tests.yaml file within the root of the Tests project determines the test mode when running tests inside Visual Studio

  • u for unit tests
  • i for integration tests
  • m for mixed mode i.e. unit and integration tests

The build script has a number of different build targets to run different types of tests, see the Targets.fsx script in the scripts project for the complete list, but the main ones are:

Compile and run unit tests

build.bat

with no target will run the Build target, compiling the solution and running unit tests

Quick Compile and run integration tests

build.bat Integrate [Elasticsearch Version Number e.g. 2.2.0]

will quick compile the solution and run integration tests against the target Elasticsearch version. The first time this is run for a version of Elasticsearch, it will download Elasticsearch and unzip Elasticsearch, install the plugins necessary to run the integration tests, and start the node. Because of this, the first run may take some time to start.

##Troubleshooting

###Could not load file or assembly FSharp.Core

You may come across an exception similar to below when running the build script

Unhandled Exception: System.IO.FileLoadException: Could not load file or assembly 'FSharp.Core, Version=4.3.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)    at <StartupCode$FAKE>.$Program.main@()

The 1.x and master branches have diverged dramatically as a result of changes in preparation for 2.0. This includes changes to the build process such that switching between the master and 1.x branches and back again can change the versions of packages used within the build processes. To rectify this issue, try deleting the packages folder within the root of the solution and run the build script again.

If working on both 1.x and 2.0 versions of NEST, it is recommended to clone the git repository for each version into separate directories to avoid the need to switch between the divergent branches.