Skip to content
This repository has been archived by the owner on Jan 2, 2024. It is now read-only.

Latest commit

 

History

History
35 lines (28 loc) · 2.26 KB

CONTRIBUTING.md

File metadata and controls

35 lines (28 loc) · 2.26 KB
Raw

Contributing to TradeOfferManager

Thanks for your interest in making TradeOfferManager better! I'd appreciate it if you read over this document quickly before you submit your issue or pull request. It'll make things go much smoother.

Issues

Submitting an issue?

  • If you're reporting a bug, please include all relevant details.
    • A descriptive title helps for one. Titles of just "Error" or "It doesn't work" really don't help.
    • Please describe what you're trying to do, what actually happens, and what you can do to reproduce the problem.
    • If you have an error message or a crash, please include the full text of the error message and the stack trace.
    • Include the relevant snippet of your code. Wrap it in ```js /* code */ ``` and GitHub will format it nicely for you.
  • If you're requesting a feature, please be descriptive and understanding.
    • A good title makes a difference. Please briefly describe what you're requesting in the title.
    • Be descriptive in the issue body, too. Say what you want to do, and ideally what the method should be named.
    • Be understanding if I don't think that your feature request falls within the scope of this module.
  • If you're asking a question or requesting support, please don't submit a GitHub issue.

Pull Requests

Submitting a pull request? Great! Thanks for contributing your time and code! Please keep the following in mind.

  • Please follow the existing code style.
    • Tabs for indentation
    • camelCase for variables and functions
    • Opening braces on the same line as the if/for/while statement
    • etc.
  • Please avoid breaking changes. If you make a breaking change that can be done in a backwards-compatible manner, I won't accept it.
  • Please don't increment the version number in package.json. I'll do that myself when I publish it to npm.
  • Please include a brief description of your change in the pull request if it's not immediately apparent from the code.
  • Be understanding if I don't think that your change falls within the scope of this module.