Skip to content

Commit

Permalink
Switch to GitHub issues
Browse files Browse the repository at this point in the history
  • Loading branch information
hazzik committed Oct 12, 2017
1 parent 4713cce commit 698e5ec
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 31 deletions.
25 changes: 11 additions & 14 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,18 +4,17 @@ This document describes the policies and procedures for working with NHibernate.

For the least friction, please follow the steps in the order presented, being careful not to miss any. There are many details in this document that will help your contribution go as smoothly as possible. Please read it thoroughly.

## Check for Existing Issues Visit
## Create or find a GitHub Issue

[https://nhibernate.jira.com][1] and search for your issue. If you see it, voting for it is a good way to increase the visibility of the issue.
## Create a JIRA Issue
GitHub is used to generate the release notes and serves as a central point of reference for all changes that have occurred to NHibernate.

JIRA is used to generate the release notes and serves as a central point of reference for all changes that have occurred to NHibernate.
Visit [https://github.com/nhibernate/nhibernate-core/issues][1] and search for your issue. If you see it, giving it a like is a good way to increase the visibility of the issue.

Before creating an issue, please do your best to verify the existence of the problem. This reduces noise in the issue tracker and helps conserve the resources of the team for more useful tasks. Note the issue number for future steps. Ex. NH-2318
Before creating an issue, please do your best to verify the existence of the problem. This reduces noise in the issue tracker and helps conserve the resources of the team for more useful tasks. Note the issue number for future steps.

## Fork and Clone from GitHub

The main GitHub repository is at <https://github.com/nhibernate/nhibernate-core>. If you plan to contribute your test case or improvement back to NHibernate, you should visit that page and fork the repository so you can commit your own changes and then submit a pull request.
The main GitHub repository is at [https://github.com/nhibernate/nhibernate-core][2]. If you plan to contribute your test case or improvement back to NHibernate, you should visit that page and fork the repository so you can commit your own changes and then submit a pull request.

## The Build Menu

Expand All @@ -38,7 +37,7 @@ The main GitHub repository is at <https://github.com/nhibernate/nhibernate-core>
2. Run the build menu and select option B to create a new test configuration. Notepad will pop up and you should edit the connection string information, saving it when you're done. These configurations will appear in the "available-test-configurations" folder.
3. Run the build menu and select option C to activate the test configuration you created. The appropriate configuration will be copied to the "current-test-configuration" folder.
4. (Optional) Run all the tests with option D and hopefully you will see no failing tests. The build may fail on certain databases; please ask on the mailing list if you are having trouble.
5. Before using the database for unit tests from Visual Studio, you'll need to create an empty database that matches your connection string. [NH-2866][2] will make this easier, but for now you just have to drop/create the database specified in your connection string yourself.
5. Before using the database for unit tests from Visual Studio, you'll need to create an empty database that matches your connection string.

## Creating a Test Case to Verify the Issue

Expand Down Expand Up @@ -74,9 +73,7 @@ counter-part, you should regenerate the async code. Use build-menu option H for

Ensure that your e-mail address and name are configured appropriately in Git.

Create a feature branch so it's easy to keep it separate from other improvements. Having a pull request accepted might involve further commits based on community feedback, so having the feature branch provides a tidy place to work from. Using the issue number as the branch name is good practice.

When you commit, please include the issue number in your commit message. This will allow the JIRA issue tracker to automatically link your commits to the issue. By example: *NH-1234 - test case for blah trouble*. Make sure you do not put any non whitespace character adjacent to the issue number, as this would prevent JIRA to match it. (Avoid *NH-1234: test case for blah trouble*.)
Create a feature branch so it's easy to keep it separate from other improvements. Having a pull request accepted might involve further commits based on community feedback, so having the feature branch provides a tidy place to work from. Using the issue number as the branch name is good practice.

## Implementing the Bug Fix or Improvement

Expand All @@ -90,21 +87,21 @@ Please note that some tests assume a case insensitive accent sensitive database

## Submit a Pull Request

Be sure to link to the JIRA issue in your GitHub pull request. Also, go back to your JIRA issue and link to the pull request.
If you are fixing an existing issue, please make sure to include this issue number in your GitHub pull request.

We use tabs for code indentation, not spaces. As this is not the default in Visual Studio, you will need to reconfigure Visual Studio to indent with tabs whenever you work on the NHibernate codebase. To make this easier, NHibernate has an [editorconfig][3] configuration file to switch Visual Studio automatically between tabs and spaces mode. It is recommended you install editorconfig from the Visual Studio Extension Manager.

After submitting your pull request, come back later to check the outcome of automated builds. If some have failed, they will be listed in your pull request with a link to the corresponding TeamCity build. Find out in the build which tests are newly failing, and take appropriate action. Some of those builds have many known failing tests, which does not trigger a build failure. A *Comparison.txt* file in build Artifacts may help finding which failing tests are not known failing tests and must be addressed.

## Further Discussion

The NHibernate team monitors JIRA and GitHub regularly, so your request will be noticed. If you want to discuss it further, you are welcome to post to the [nhibernate-development mailing list][4].
The NHibernate team monitors GitHub regularly, so your request will be noticed. If you want to discuss it further, you are welcome to post to the [nhibernate-development mailing list][4].

## Happy Contributing!

The NHibernate community values your contributions. Thank you for the time you have invested.

[1]: https://nhibernate.jira.com/
[2]: https://nhibernate.jira.com/browse/NH-2866
[1]: https://github.com/nhibernate/nhibernate-core/issues/
[2]: https://github.com/nhibernate/nhibernate-core/
[3]: http://www.editorconfig.org/
[4]: http://groups.google.com/group/nhibernate-development
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,11 +36,11 @@ There are two official NHibernate community forums:
Bug Reports
-----------

If you find any bugs, please report them using the [JIRA bug tracker][C1]. A
If you find any bugs, please report them using the [GitHub issue tracker][C1]. A
test-case that demonstrates the issue is usually required. Instructions on providing a test-case
can be found [here][C2].

[C1]: http://nhibernate.jira.com
[C1]: http://github.com/nhibernate/nhibernate-core/issues
[C2]: http://nhibernate.info/blog/2008/10/03/the-best-way-to-solve-nhibernate-bugs-submit-good-unit-test.html

Licenses
Expand Down
21 changes: 7 additions & 14 deletions ReleaseProcedure.txt
Original file line number Diff line number Diff line change
@@ -1,12 +1,11 @@
These are the tasks typically needed to create an official NHibernate release.


* In Jira, make sure no open issues have "fix version" set to the release
* In GitHub, make sure no open issues have milestone set to the release
that is about to happen (as these will show in the release notes).

* Update releasenotes.txt with list of issues from Jira's Release Notes
feature. See if any additions to the list of "known breaking changes"
are necessary.
* Update releasenotes.txt with list of issues. See if any additions to the
list of "known breaking changes" are necessary.

* Check/update version number in build-common/common.xml.

Expand All @@ -24,7 +23,7 @@ These are the tasks typically needed to create an official NHibernate release.

* If no error or other issue requiring further changes occurred, tag the
current git HEAD with a tag according to version number. Don't forget to
push the tag to github.
push the tag to github.

* Upload binary and source zip files to SourceForge. Update the "latest
release".
Expand All @@ -35,17 +34,11 @@ These are the tasks typically needed to create an official NHibernate release.
should already be available in the build directory, together with a batch
file to push them.

* In Jira, mark the version as released.
* In GitHub, mark the milestone as released.

* Post release announcement to nhusers, nhibernate-development and as
project news on SourceForge.



* If this was a stable branch, merge it forward to master. Perhaps some
changes need to be edited out in the merge, but this will reduce the
risk of some issue being left unfixed on master.


* Several weeks or months after the release, in Jira, we will assume all
issues resolved in the release have been verified. Go ahead and close
them.
risk of some issue being left unfixed on master.
2 changes: 1 addition & 1 deletion doc/reference/modules/preface.xml
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@
<para>
If you have questions, use the
<ulink url="https://groups.google.com/forum/#!forum/nhusers">NHibernate user forum</ulink>.
We also provide a <ulink url="https://nhibernate.jira.com/">JIRA issue tracking system</ulink>
We also provide a <ulink url="https://github.com/nhibernate/nhibernate-core/issues">GitHub issue tracking system</ulink>
for bug reports and feature requests.
If you are interested in the development of NHibernate, join the developer mailing list.
If you are interested in translating this documentation into your language, contact us
Expand Down

0 comments on commit 698e5ec

Please sign in to comment.