-
-
Notifications
You must be signed in to change notification settings - Fork 658
Contributing
Robin Rodricks edited this page Dec 17, 2023
·
4 revisions
First you must "fork" FluentFTP, then make changes on your local version, then submit a "pull request" to request me to merge your changes. To do this:
- Click Fork on the top right of this page
- Open your version here : https://github.com/YOUR_GITHUB_USERNAME/FluentFTP
- Download Github Desktop and login to your account
- Click File > Clone repository... and select FluentFTP in the list
- Edit "Local path" with the folder where you want to place the source code and press Clone
- Edit the project using Visual Studio 2017 Community or better
- Switch to Github Desktop, which should now show a list of the changed files
- Type a Commit Summary (bottom left), and click Commit
- Click Push to origin (top right)
- Open the pull requests page to create a PR
- Click New pull request (top right)
- Click compare across forks (blue link, top right)
- On the right "head fork" select the fork with your username
- Click Create pull request
- Summarize the changes you made in the title
- Type details about the changes you made in the description
- Click Create pull request
- Thank you!
- A valid fix or feature implementation
- Alpha testing performed by the contributor shows a positive change or successful fix
- All automated tests must pass (unit & integration)
- Follows C# best practices and industry conventions
- Clean and structured code
- Follows FluentFTP conventions in regard to source code organization structure and naming scheme
- Only API methods usable by end users must be
public
, everything else must beprivate
orinternal
- Important or complicated sections must be documented (commented in the code itself)
- Add references to FluentFTP issue tickets where possible to show which issue was catered to
- Auto Connection
- Auto Reconnection
- FTP(S) Connection
- FTP(S) Connection using GnuTLS
- FTPS Proxies
- Custom Servers
- Custom Commands
- v40 Migration Guide