Thanks for taking the time to contribute!
This project is simpler than most, so it’s a good place to start contributing to the open source community, even if you’re a newbie.
We are accepting these sorts of changes and requests:
- Bug reports and fixes
- Usability improvements
- Documentation updates
- New reputable “by the book” indicators and overlays
We are not accepting things that should be done in your own wrapper code:
- Personal customizations and preferences
- Modified or augmented outputs that are not intrinsic
If you have general interest in contributing, but are not sure where to start, please contact us and we can help to find work in an area of interest.
Reporting bugs and feature requests
If you suspect a problem, please report a bug Issue with a detailed description of the problem, steps to reproduce, code samples, and any reference materials. For enhancements, create a feature Issue.
Use the Discussions area for general ideation and help/usage questions.
- Planned work is managed in the backlog.
- Work items are primarily entered as Notes (not Issues), except where an issue or feature is user reported. With that said, Notes can be converted to Issues if in-progress and collaborative discussion is needed.
- Read this first: A Step by Step Guide to Making Your First GitHub Contribution. I also have a discussion on Forking if you have questions.
- If you want to work on something specific, please mention your intention on the related Issue. If an Issue does not exist for your contribution, please create one before starting. This will help us reserve that feature and avoid duplicative efforts.
- If you are adding a new indicator, the easiest way to do this is to copy the folder of an existing indicator and rename everything using the same naming conventions and taxonomy. All new indicators should include tests.
- Do not commingle multiple contributions on different topics. Please keep changes small and separate.
- Review the
tests/indicatorsfolder for examples of unit tests. Just copy one of these.
- New indicators should be tested against manually calculated, proven, accurate results. It is helpful to include your manual calculations spreadsheet in the appropriate indicator test folder when submitting changes.
- Historical Stock Quotes are automatically added to unit test methods. A
Data.Quotes.xlsxExcel file is included in the
tests/_commonfolder that is an exact copy of what is used in the unit tests. Use a copy of this file for your manual calculations to ensure that it is correct. Do not commit changes to the original file.
- We expect all unit tests to execute successfully and all Errors and Warning resolved before you submit your code.
- Failed builds or unit testing will block acceptance of your Pull Request when submitting changes.
Tests.Performance console application in
Release mode will produce benchmark performance data that we include on our documentation site.
# run all performance benchmarks dotnet run -c Release # run individual performance benchmark dotnet run -c Release --filter *.GetAdx
This site uses GitHub Pages and Jekyll construction with Front Matter. The documentation site is in the
docs folder. Build the site locally to test that it works properly. See GitHub documentation for initial Ruby and Jekyll setup instructions for Github Pages.
# from /docs folder bundle install bundle exec jekyll serve -o -l # the site will open http://127.0.0.1:4000
When adding or updating indicators:
- Add or update the
- Page image assets go here:
/docs/assets/and can be optimized to
webpformat using ImageMagick or the cwebp Encoder CLI and a command like
cwebp -resize 832 0 -q 100 examples.png -o examples-832.webp
- use Lighthouse in Chrome, or
- build the site locally (see above), then:
npx pa11y-ci --sitemap http://127.0.0.1:4000/sitemap.xml"
By submitting changes to this repo you are also acknowledging and agree to the terms in both the Developer Certificate of Origin (DCO) 1.1 and the Apache 2.0 license. These are standard open-source terms and conditions.
When ready, submit a Pull Request with a clear description of what you’ve done and why it’s important. Always write a clear log message for your commits. One-line messages are fine for most changes.
After a Pull Request is reviewed, accepted, and [squash] merged to
main, we may batch changes before publishing a new package version to the public NuGet repository. Please be patient with turnaround time.
Code reviews and administration
If you want to contribute administratively, do code reviews, or provide general user support, we’re also currently seeking a few core people to help. Please contact us if interested.
Standards and design guidelines
- Guiding principles for this project
- .NET Framework Design Guidelines
- NuGet Best Practices
- Semantic Version 2.0
We use the
GitVersion tool for semantic versioning. It is mostly auto generated in the Azure DevOps build.
|Major|| ||A significant deviation with major breaking changes.|
|Minor|| ||A new feature, usually new non-breaking change, such as adding an indicator. Minor breaking changes may occur here and are denoted in the release notes.|
|Patch|| ||A small bug fix, chore, or documentation change.|
|Increment|| ||Intermediate commits between releases.|
This only needs to be done on the merge to
main when the Pull Request is committed, so your feature branch does not need to include this as it will get squashed anyway.
+semver: majoras a PR merge commit message will increment the major x.-.- element
+semver: minoras a PR merge commit message will increment the minor -.x.- element
+semver: patchas a PR merge commit message will increment the minor -.-.x element. Patch element auto-increments, so you’d only need to do this to override the next value.
tag, in accordance with the above schema, is introduced automatically after deploying to the public NuGet package manager and is reflected in the Releases.
This repository uses a standard Apache 2.0 open-source license. It enables open-source community development by protecting the project and contributors from certain legal risks while allowing the widest range of uses, including in closed source software. Please review the license before using or contributing to the software.
Start a new discussion or submit an issue if it is publicly relevant. You can also direct message @daveskender.
Thanks, Dave Skender