First off, thank you for considering contributing to xctools. It's people like you that make xctools such a great tool.
Following these guidelines helps to communicate that you respect the time of the developers managing and developing this open source project. In return, they should reciprocate that respect in addressing your issue, assessing changes, and helping you finalize your pull requests.
xctools is an open source project and we love to receive contributions from our community — you! There are many ways to contribute, from writing tutorials or blog posts, improving the documentation, submitting bug reports and feature requests or writing code which can be incorporated into xctools itself.
- Ensure that changes in the project are tested.
- Before merging, all the checks on GitHub should pass.
- Create issues for any major changes and enhancements that you wish to make. Discuss things transparently and get community feedback.
- Don't add any classes to the codebase unless absolutely needed.
- Keep feature versions as small as possible, preferably one new feature per version.
- Be welcoming to newcomers and encourage diverse new contributors from all backgrounds.
- Comply the code of conduct when you participate in the project.
Unsure where to begin contributing to xctools? You can start by looking through these easy and thread issues:
- difficulty:easy - issues which should only require a few lines of code, and a test or two.
- type:thread - issues which should be a bit more involved than beginner issues.
Both issue lists are sorted by total number of comments. While not perfect, number of comments is a reasonable proxy for impact a given change will have.
Working on your first Pull Request? You can learn how from this free series, How to Contribute to an Open Source Project on GitHub.
At this point, you're ready to make your changes! Feel free to ask for help; everyone is a beginner at first 😸
- Create your own fork of the code
- Do the changes in your fork
- If you like the change and think the project could use it:
- Be sure you have followed the code style for the project.
- Make sure
bundle exec rake ci
passes.
As a rule of thumb, changes are obvious fixes if they do not introduce any new functionality or creative thinking. As long as the change does not affect functionality, some likely examples include the following:
- Spelling / grammar fixes
- Typo correction, white space and formatting changes
- Comment clean up
- Bug fixes that change default return values or error codes stored in constants
- Adding logging messages or debugging output
- Changes to ‘metadata’ files like Gemfile, .gitignore, build scripts, etc.
- Moving source files from one directory or package to another
If you find a security vulnerability, do NOT open an issue. Email [email protected] instead.
If you don’t want to use your personal contact information, set up a “security@” email address. Larger projects might have more formal processes for disclosing security, including encrypted communication.
When filing an issue, make sure to answer these five questions:
- What version of Xcode/Swift are you using?
- What operating system and processor architecture are you using?
- What did you do?
- What did you expect to see?
- What did you see instead?
Before suggesting a new feature, make sure that it aligns with the project philosophy, providing command line tools to automate Xcode-related tasks.
If you find yourself wishing for a feature that doesn't exist in xctools, you are probably not alone. There are bound to be others out there with similar needs. Many of the features that xctools has today have been added because our users saw the need. Open an issue on our issues list on GitHub which describes the feature you would like to see, why you need it, and how it should work.
The core team looks at Pull Requests on a regular basis.
After feedback has been given we expect responses within two weeks. After two weeks we may close the pull request if it isn't showing any activity.
You can chat with the core team on Slack. You can join the group on the following link.
- Commit message should include the issue number together with the commit message:
[#number] commit message
.