We love your input! We want to make contributing to Enterprise IT Administration Scripts as easy and transparent as possible, whether it's:
- Reporting a bug
- Discussing the current state of the code
- Submitting a fix
- Proposing new features
- Becoming a maintainer
We use Github to host code, to track issues and feature requests, as well as accept pull requests.
We Use Github Flow
Pull requests are the best way to propose changes to the codebase. We actively welcome your pull requests:
- Fork the repo and create your branch from
main
. - If you've added code that should be tested, add tests.
- If you've changed APIs, update the documentation.
- Ensure the test suite passes.
- Make sure your code follows our coding standards.
- Issue that pull request!
In short, when you submit code changes, your submissions are understood to be under the same MIT License that covers the project. Feel free to contact the maintainers if that's a concern.
Report bugs using Github's issue tracker
We use GitHub issues to track public bugs. Report a bug by opening a new issue; it's that easy!
Great Bug Reports tend to have:
- A quick summary and/or background
- Steps to reproduce
- Be specific!
- Give sample code if you can.
- What you expected would happen
- What actually happens
- Notes (possibly including why you think this might be happening, or stuff you tried that didn't work)
- Use consistent indentation (preferably spaces)
- Use meaningful variable and function names
- Include comments for complex logic
- Keep functions focused and modular
- Write self-documenting code where possible
- Follow PowerShell Best Practices and Style Guide
- Use approved verbs for function names
- Include comment-based help for functions
- Use proper error handling with try/catch blocks
- Follow naming conventions (PascalCase for functions, camelCase for variables)
- Follow PEP 8
- Use virtual environments
- Include requirements.txt or setup.py
- Write docstrings for functions and classes
- Use type hints where appropriate
- Follow Google's Shell Style Guide
- Make scripts executable
- Include shebang line
- Use shellcheck for validation
- Include usage information in comments
-
README Files
- Each script should have its own README
- Include purpose, usage, and examples
- List dependencies and prerequisites
- Provide configuration instructions
-
Code Comments
- Document complex algorithms
- Explain non-obvious technical decisions
- Include references to relevant documentation
-
Function Documentation
- Document parameters and return values
- Include usage examples
- Note any side effects
-
Unit Tests
- Write tests for new functionality
- Maintain existing tests
- Ensure tests are meaningful
-
Integration Tests
- Test interaction between components
- Verify system-level functionality
- Include environment setup instructions
-
Test Documentation
- Document test coverage
- Explain test scenarios
- Include test data explanation
By contributing, you agree that your contributions will be licensed under its MIT License.
This document was adapted from the open-source contribution guidelines for Facebook's Draft.