Skip to content

Latest commit

 

History

History
33 lines (17 loc) · 3.88 KB

CONTRIBUTING.md

File metadata and controls

33 lines (17 loc) · 3.88 KB

How to contribute to AutoFixture

AutoFixture is currently being developed in C# on .NET 4 using Visual Studio 2010/2/3 with xUnit.net as the unit testing framework. So far, all development has been done with TDD, so there's a pretty high degree of code coverage, and the aim is to to keep it that way.

Dependencies

All binaries (such as xUnit.net) are included as NuGet packages in the source control repository under the \Packages folder. All additional binaries not part of .NET 4 must go there as well, so that it would be possible to pull down the repository and immediately be able to compile and run all tests.

Verification

There are several different targeted solutions to be found under the \Src folder, but be aware that the final verification step before pushing to the repository is to successfully run all the unit tests in the BuildRelease.ps1 file.

As part of the verification build, Visual Studio Code Analysis is executed in a configuration that treats warnings as errors. No CA warnings should be suppressed unless the documented conditions for suppression are satisfied. Otherwise, a documented agreement between at least two active developers of the project should be reached to allow a suppression of a non-suppressible warning.

Pull requests

When developing for AutoFixture, please respect the coding style already present. Look around in the source code to get a feel for it.

Please keep line lengths under 120 characters. Line lengths over 120 characters don't fit into the standard GitHub code listing window, so it requires vertical scrolling to review.

Also, please follow the Open Source Contribution Etiquette. AutoFixture is a fairly typical open source project: if you want to contribute, start by creating a fork and sending a pull request when you have something you wish to commit. When creating pull requests, please keep the Single Responsibility Principle in mind. A pull request that does a single thing very well is more likely to be accepted. See also the article The Rules of the Open Road for more good tips on working with OSS and Pull Requests, as well as these 10 tips for better Pull Requests.

For complex pull requests, you are encouraged to first start a discussion on the issue list. This can save you time, because the AutoFixture regulars can help verify your idea, or point you in the right direction.

Some existing issues are marked with the jump-in tag. These are good candidates to attempt, if you are just getting started with AutoFixture.

When you submit a pull request, you can expect a response within a day or two. We (the maintainers of AutoFixture) have day jobs, so we may not be able to immediately review your pull request, but we do make it a priority. Also keep in mind that we may not be in your time zone.

Most likely, when we review pull requests, we will make suggestions for improvement. This is normal, so don't interpret it as though we don't like your pull request. On the contrary, if we agree on the overall goal of the pull request, we want to work with you to make it a success.

Continuous Delivery

AutoFixture is set for Continuous Integration on http://teamcity.codebetter.com/project.html?projectId=project129&tab=projectOverview. The latest build (including strong names) can be downloaded from there, but Continuous Delivery is also enabled, which means that the latest build of master is also automatically published on CodePlex and NuGet.