You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: CONTRIBUTING.md
+32-17
Original file line number
Diff line number
Diff line change
@@ -11,19 +11,25 @@ This project accepts contributions in various languages. Not only development b
11
11
12
12
## English
13
13
14
+
### Setup (en)
15
+
16
+
[Eask] is required to build the package. You need to install it using node.js and npm, or [download a prebuilt executable][eask-releases]. See [Eask Introduction][eask-introduction] and [Install Eask][eask-install] for more details.
17
+
18
+
You can run the tests using the following command:
19
+
20
+
```sh
21
+
make test
22
+
```
23
+
14
24
### Guideline (en)
15
25
16
-
All contributions to PHP Mode are welcome. But please try to do the following when sending improvements or bug fixes:
26
+
All contributions to PHP Mode are welcome. But please try to do the following when submitting enhancements or bug fixes:
17
27
18
-
1. Add your name to the list of ‘Contributors’ in this `README.md` file if it is not there already. If you have a GitHub page and/or personal site then please feel free to link your name to it so people can see your other work.
19
-
2. If your contribution addresses an issue on the GitHub project page then include a single line like `GitHub-Issue: #16` with the appropriate issue number.
20
-
3. Make sure to update the constant `php-mode-modified`*only if you patch affects `php-mode.el`,* which means this step is unnecessary for patches related to unit tests.
21
-
4. However, please do not modify `php-mode-version-number`. The maintainers will decide what constitutes a bump in the version number.
22
-
5. Open the `php-mode-test.el` file and [run all of the tests] to ensure they still pass as expected. Sometimes we expect for a test to fail, and those unit tests have the appropriate configuration so their failure will not raise any warnings. You can use `make test` script to run all tests from a terminal, which is also useful in conjunction with [`git bisect run`].
23
-
6. Send us a pull request here on GitHub.
24
-
7. Please make your commit messages as detailed as possible. It is better to be too verbose than to write too little. Look at the commits of the maintainers to see many examples of the level of detail that we feel is ideal. Please never assume that your patch is so simple that future developers will be able to understand the *reason* for the change without comment. And that is important: your commit message should always strive to answer *"Why"* the patch exists, *"What*" does it accomplish? The maintainers will sometimes write detailed commit messages for pull-requests by other developers, but please do not rely on us to do this consistently.
28
+
1. Open `php-mode-test.el` and run all the tests to make sure they pass as expected. You can also check this by running `make test` from the terminal.
29
+
2. Please commit with a concise and clear message that effectively achieves a simple purpose.
30
+
3. Submit a pull request here on GitHub.
25
31
26
-
If you are fixing a bug related to a GitHub issue, then first of all, thank you for the help improving PHP Mode. Second, there is a `tests/` directory which contains PHP scripts for issues (although not all of them). Please consider adding a test script to that directory that documents the expected behavior and provides code that allows others to see if said behavior works properly. Then create a unit test within `php-mode-test.el` using [ERT]. Please try to follow the format of the existing tests.
32
+
If you are fixing a bug related to a GitHub issue, then first of all, thank you for helping to improve PHP Mode. Second, there is a `tests/` directory that contains PHP scripts for issues (though not all). Please consider adding a test script to that directory that documents the expected behavior and provides code that allows others to see if that behavior works properly. Then create a unit test within `php-mode-test.el' using [ERT]. Try to follow the format of the existing tests.
0 commit comments