DeveloperGuide: Difference between revisions

From The Battle for Wesnoth Wiki
Esr (talk | contribs)
Remove Subversion-specific terminology
Iceiceice (talk | contribs)
add link to git crash course
Line 1: Line 1:
For a crash course in using git / github for wesnoth development, see [[Git_for_Wesnoth_Crash_Course]].
== Commits ==
== Commits ==
* Setting up developer access to the repository: [[WesnothRepository#Commit_access]]
* Setting up developer access to the repository: [[WesnothRepository#Commit_access]]

Revision as of 04:36, 27 February 2014

For a crash course in using git / github for wesnoth development, see Git_for_Wesnoth_Crash_Course.

Commits

  • Setting up developer access to the repository: WesnothRepository#Commit_access
  • Register to commit mailing list: https://mail.gna.org/listinfo/wesnoth-commits or get the list moderator to approve commit messages from you otherwise
  • Trunk should compile and work after commit
  • Few small commits are better than a big one (hard to review), so when possible split it in working parts with info about where you are going
  • Be on #wesnoth-dev IRC channel and coordinate with other developers. Bots report commits and some devs may ask you a question about it.
  • Don't forget changelogs: changelog for any (significant enough) changes and player_changelog for changes visible to users.
  • Always check your changes before commit (see WesnothRepository#Reviewing your changes).
  • Commit message:
    • Mention any change, especially if some are unrelated to the main one (but you should use separated commits for this).
    • Mention "bug #1234" for automatic cc to that gna bug number
    • Mention a developer's IRC name will ping him on IRC (when the bot report it), and if he's not there, he may see it on the IRC logs

Bugs management

  • Change status of fixed bugs to "Fixed" when committed
  • Change status of fixed bugs to "Closed" when released, see ReportingBugs#Bug_protocol for details
  • Check if there is new bugs relevant to your code and if any, assign them to you.

See also

This page was last edited on 27 February 2014, at 04:36.
Cookies help us deliver our services. By using our services, you agree to our use of cookies.