Your input is welcome! Contributing to this project is as easy as:
- Reporting a bug
- Discussing the current state of the code
- Submitting a fix
- Proposing new features
- Creating a new report
When contributing to this repository, please first discuss the change you wish to make via issue, or direct message in the PowerShell Slack channel before making a change.
This project uses 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.
A good quality pull request will have the following characteristics:
- It will be a complete piece of work that adds value in some way.
- It will have a title that reflects the work within, and a summary that helps to understand the context of the change.
- There will be well written commit messages, with well crafted commits that tell the story of the development of this work.
- Ideally it will be small and easy to understand. Single commit PRs are usually easy to submit, review, and merge.
- The code contained within will meet the best practices set by the team wherever possible.
- Fork this repository.
- Add
https://github.com/tpcarman/As-Built-Report.git
as a remote namedupstream
.git remote add upstream https://github.com/tpcarman/As-Built-Report.git
- Create your feature branch from
dev
. - Work on your feature.
- Update CHANGELOG.md with add / remove / change information
- Update README.md with any new information, such as features, instructions, parameters and/or examples
- Squash commits into one or two succinct commits.
git rebase -i HEAD~n
# n being the number of previous commits to rebase
- Ensure that your branch is up to date with
upstream/dev
.git checkout <branch>
git fetch upstream
git rebase upstream/dev
- Push branch to your fork.
git push --force
- Open a Pull Request against the
dev
branch of this repository.
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.
GitHub issues is used to track issues and 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)
Code contributors should follow the PowerShell Guidelines wherever possible to ensure scripts are consistent in style.
Use PSScriptAnalyzer to check code quality against PowerShell Best Practices.
- Use #requires statements in all report scripts to ensure Windows PowerShell version, modules, snap-ins, and module and snap-in version prerequisites are met.
- Use PascalCasing for all public member, type, and namespace names consisting of multiple words.
- Keep the number of required PowerShell modules to 2 per script.
- Use custom label headers within tables, where required, to make easily readable labels.
- Favour readability over brevity
- Do not include code within report script to install or import PowerShell modules. Instead, use #requires statements to ensure Windows PowerShell version, modules, snap-ins, and module and snap-in version prerequisites are met.
By contributing, you agree that your contributions will be licensed under its MIT License.