]> git.ipfire.org Git - thirdparty/bulma.git/commitdiff
Document how to try the changes (#1920)
authorAna María Martínez Gómez <ammartinez@suse.de>
Tue, 5 Jun 2018 23:45:21 +0000 (01:45 +0200)
committerJeremy Thomas <bbxdesign@gmail.com>
Tue, 5 Jun 2018 23:45:21 +0000 (00:45 +0100)
* Document how to try the changes

Add to the contributing guide how to rebuild the css and where the
documentation to try the documentation changes is.

* Remove Bulma dependencies installation form PR template

Install Bulma dependencies is not needed to send a PR.

* Add how to try your changes to PR templates

And after that ask if you tried the changes.

.github/CONTRIBUTING.md
.github/PULL_REQUEST_TEMPLATE.md

index f37b30c34184817b6c90083f1b3a4bd833890ef7..f154d95a29f9948dda6feaa4f8d8a64ab68519cd 100644 (file)
@@ -13,6 +13,17 @@ Thanks for helping out! 😇
 * Limit the first line to 72 characters or fewer
 * Reference issues and pull requests liberally
 
+### Try your changes
+
+When modifying any `.sass`, you will need to rebuild the css. You can do this by running:
+
+```
+npm install
+npm run build
+```
+
+To see documentation changes check [the documentation readme](docs/README.md).
+
 ### Bulma Sass styleguide
 
 * **No semi-colons** `;` or **curly braces** `{` `}`
index 91c9a15bd3e3115091c10513622cfac4df71d70d..4a92110f6d400fd3bb3efbbd0746df8111140233 100644 (file)
@@ -16,13 +16,14 @@ This is a **new feature | improvement | bugfix | documentation fix**.
 <!-- Think of performance, build time, usability, complexity, coupling…) -->
 
 ### Testing Done
-<!-- How have you confirmed this feature works? -->
-<!-- Please explain more than "Yes". -->
 
 <!-- BEFORE SUBMITTING YOUR PR, MAKE SURE TO FOLLOW THESE STEPS: -->
 <!-- 1. Pull the latest `master` branch -->
-<!-- 2. Run `npm install` to install all Bulma dependencies -->
-<!-- 3. Make sure your Sass code is compliant with the [Bulma Sass styleguide](https://github.com/jgthms/bulma/blob/master/.github/CONTRIBUTING.md#bulma-sass-styleguide) -->
-<!-- 4. Make sure your PR only affects `.sass` or documentation files -->
+<!-- 2. Make sure your Sass code is compliant with the [Bulma Sass styleguide](https://github.com/jgthms/bulma/blob/master/.github/CONTRIBUTING.md#bulma-sass-styleguide) -->
+<!-- 3. Make sure your PR only affects `.sass` or documentation files -->
+<!-- 4. [Try your changes](https://github.com/jgthms/bulma/blob/master/.github/CONTRIBUTING.md#try-your-changes). -->
+
+<!-- How have you confirmed this feature works? -->
+<!-- Please explain more than "Yes". -->
 
 <!-- Thanks! -->