Skip to content

Commit 6243d40

Browse files
authored
Created contribution guidelines
1 parent 3c40838 commit 6243d40

File tree

1 file changed

+97
-0
lines changed

1 file changed

+97
-0
lines changed

CONTRIBUTING.md

+97
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,97 @@
1+
# Contributing to AssistantComputerControl
2+
3+
Thanks for your interest in helping the project!
4+
5+
The following is a set of guidelines for contributing to the AssistantComputerControl project. These are mostly guidelines, not rules.
6+
7+
#### Table Of Contents
8+
9+
[How Can I Contribute?](#how-can-i-contribute)
10+
* [Reporting Bugs](#reporting-bugs)
11+
* [Suggestings](#suggestions)
12+
* [Pull Requests (code contribution)](#pull-requests-code-contribution)
13+
14+
Additional
15+
* [Code Styleguide](#code-styleguide)
16+
17+
## Reporting bugs
18+
Found a bug? Let's get it fixed! For us to get the issue resolved as quick as possible, we need all the information you can provide us.
19+
20+
:heavy_exclamation_mark: **Before you open an issue** here on the project page, please take the time to check [the issue list](https://github.com/AlbertMN/AssistantComputerControl/issues?utf8=%E2%9C%93&q=is%3Aissue) to see if the problem you're experiencing has already been reported. If your issue is somewhat like another issue, but not completely the same, include a link to the issue in the issue you create.
21+
22+
Has the issue you're having has not been reported yet? Then [open a new issue](https://github.com/AlbertMN/AssistantComputerControl/issues/new) and describe the issue in as much details as you can.
23+
24+
**General details to include in your issue report;**
25+
- When does the issue occour? _(how to make it happen)_
26+
- How often does it happen?
27+
- Which version of the software are you on?
28+
- Did this happen in the last update as well?
29+
30+
**You should also attach any ACC log files**, located on your computer at; `Documents/AssistantComputerControl/`. This folder can contain a `log.txt` and `error_log.txt` file - attach both if they are present.
31+
32+
Once you feel like you have described your issue as well as possible, go ahead and create the issue, and wait for a reponse. Issues are often responded to within one day and resolved in a matter of days as well.
33+
How fast the issue is resolved also heavily depends on your cooperation. Often times the developer(s) can't replicate the issue, and in this case you will be asked to test new versions of the software to see if the problem has been resolved.
34+
35+
## Suggestions
36+
All suggestions are welcome! Most of what the software can has been added as a result of user suggestions. We prefer to keep suggestions in the [Discord server](https://discord.gg/B9YGPNF), in the `#suggestions` channel.
37+
38+
## Pull Requests (code contribution)
39+
Want to contribute to the project by coding a new feature, fixing a bug or generally just improve the project? Your help is very welcome!
40+
41+
Seeing the software has tens of thousands of users using it every day, the accepted code-contributions however _does not_ include;
42+
- Major rewrites of entire files
43+
- _To keep the project manageable, secure and reliable, development of large scale functionality that isn't a new action won't be accepted_
44+
- Changes to the way core parts of the software works
45+
- _Fx the way we communicate with the cloud services or IFTTT_
46+
47+
All other contributions are very welcome! Please refer to the [Code Styleguide](#code-styleguide) before writing code for the project, to ensure your code matches the style of the project.
48+
49+
____
50+
51+
## Code Styleguide
52+
Before making code contributions to the project, there are a few "rules" regarding the way the code looks.
53+
54+
**General info**
55+
- Our brace indentation standard is [K&R](https://en.wikipedia.org/wiki/Indentation_style#K&R_style) _(braces on same line as the statement)_
56+
- We use `tabs` - not spaces.
57+
- The project is a Visual Studio project running on .NET Framework 4.6
58+
- The primary language is C#, only other programming language used is JavaScript and a bit of
59+
60+
:white_check_mark: **ACC coding style;**
61+
```
62+
if (condition) {
63+
//code
64+
} else {
65+
66+
}
67+
68+
class x {
69+
//
70+
}
71+
var variable1 = "",
72+
variable2 = "",
73+
variable3 = "";
74+
```
75+
76+
:x: **"Wrong" coding style**
77+
78+
_Not that this codings tyle is "wrong", it's just not the style used in this particular project - it's a matter of taste._
79+
```
80+
if (condition)
81+
{
82+
//code
83+
}
84+
else
85+
{
86+
87+
}
88+
89+
class x
90+
{
91+
//
92+
}
93+
var variable1 = "", variable2 = "", variable3 = "";
94+
```
95+
96+
If you have any questions about something else regarding the coding style, all you have to do is take a look at the code, and do like it's done there.
97+

0 commit comments

Comments
 (0)