Skip to content

Latest commit

 

History

History
152 lines (116 loc) · 7.47 KB

CONTRIBUTING.md

File metadata and controls

152 lines (116 loc) · 7.47 KB

Contributing to the Project

This document is the single source of truth on contributing towards this codebase. Feel free to browse the open issues and file new ones - all feedback is welcome!

Navigation

Prerequisites

When contributing to this repository, first discuss the issue with a repository maintainer via GitHub issue, Slack message, or email.

Contributor License Agreement

We only accept pull requests submitted from:

Code of Conduct

All contributors are expected to read our Code of Conduct and observe it in all interactions involving this project.

Contribution Workflow

Help is always welcome! For example, documentation can always use improvement. There's always code that can be clarified, functionality that can be extended, and tests to be added to guarantee behavior. If you see something you think should be fixed, don't be afraid to own it.

Bug reports and feature requests

Have ideas on improvements? See something that needs work? While the community encourages everyone to contribute code, it is also appreciated when someone reports an issue. Please report any issues or bugs you find through our issue tracker.

If you are reporting a bug, please include:

  • Your operating system name and version
  • Details about your local setup that might be helpful in troubleshooting (e.g. Python interpreter version, Ansible version, etc.)
  • Detailed steps to reproduce the bug

We'd also like to hear your feature suggestions. Feel free to submit them as issues by:

  • Explaining in detail how they should work
  • Keeping the scope as narrow as possible. This will make it easier to implement

Fixing issues

Look through our issue tracker to find problems to fix! Feel free to comment and tag corresponding stakeholders or full-time maintainers of this project with any questions or concerns.

Pull requests

A pull request informs the project's core developers about the changes you want to review and merge. Once you submit a pull request, it enters a stage of code review where you and others can discuss its potential modifications and add more commits later on.

To learn more, see Proposing changes to your work with pull requests in the GitHub Help Center.

To make a pull request against this project:

  1. Fork the docker-splunk GitHub repository.
  2. Clone your fork and create a branch off of develop.
    # Create a local copy (or clone) of the repository
    $ git clone [email protected]:YOUR_GITHUB_USERNAME/docker-splunk.git
    $ cd docker-splunk
    
    # Create your feature/bugfix branch
    $ git checkout -b your-branch-name develop
    
  3. Run tests to verify your environment.
    $ cd docker-splunk
    $ make test
    
  4. Push your changes once your tests have passed.
    # Add the files to the queue of changes
    $ git add <modified file(s)>
    
    # Commit the change to your repo with a log message
    $ git commit -m "<helpful commit message>"
    
    # Push the change to the remote repository
    $ git push
    
  5. Submit a pull request through the GitHub website using the changes from your forked codebase

Code Review

There are two aspects of code review: giving and receiving.

A PR is easy to review if you:

  • Follow the project coding conventions.
  • Write good commit messages, concise and descriptive.
  • Break large changes into a logical series of smaller patches. Patches individually make easily understandable changes, and in aggregate, solve a broader issue.

Reviewers are highly encouraged to revisit the Code of Conduct and must go above and beyond to promote a collaborative, respectful community.

When reviewing PRs from others, The Gentle Art of Patch Review suggests an iterative series of focuses, designed to lead new contributors to positive collaboration without inundating them initially with nuances:

  • Is the idea behind the contribution sound?
  • Is the contribution architected correctly?
  • Is the contribution polished?

Merge requirements for this project:

  • at least 2 approvals
  • a passing build from our continuous integration system.

Any new commits to an open pull request will automatically dismiss old reviews and trigger another build.

Testing

Testing is the responsibility of all contributors. In general, we try to adhere to Google's test sizing philosophy when structuring tests.

There are multiple types of tests. The location of the test code varies with type, as do the specifics of the environment needed to successfully run the test.

  1. Small: Very fine-grained; exercises low-level logic at the scope of a function or a class; no external resources (except possibly a small data file or two, but preferably no file system dependencies whatsoever); very fast execution on the order of seconds

    $ make small-tests
    
  2. Medium: Exercises interaction between discrete components; may have file system dependencies or run multiple processes; runs on the order of minutes

    $ make medium-tests
    
  3. Large: Exercises the entire system, end-to-end; used to identify crucial performance and basic functionality that will be run for every code check-in and commit; may launch or interact with services in a data center, preferably with a staging environment to avoid affecting production

    $ make large-tests
    

Continuous integration will run all of these tests either as pre-submits on PRs, post-submits against master/release branches, or both.

Documentation

We can always use improvements to our documentation! Anyone can contribute to these docs, whether you identify as a developer, an end user, or someone who just can’t stand seeing typos. What exactly is needed?

  1. More complementary documentation. Have you found something unclear?
  2. More examples or generic templates that others can use.
  3. Blog posts, articles and such – they’re all very appreciated.

You can also edit documentation files directly in the GitHub web interface, without creating a local copy. This can be convenient for small typos or grammar fixes.

Maintainers

If you need help, tag one of the active maintainers of this project in a post or comment. We'll do our best to reach out to you as quickly as we can.

# Active maintainers marked with (*)

(*) Nelson Wang - admin
(*) Tony Lee
(*) Alisha Mayor
(*) Hendo Lim
(*) Jack Meixensperger
(*) James Rigassio
(*) Mike Dickey
( ) Brent Boe
( ) Jonathan Vega
( ) Brian Bingham