- (if working in a group) give each member write access to the repo
- Turn on GitHub Pages and put a link to your website in the repo's description
- in the Branches section of your repo's settings make sure:
- The repository requires a review before pull requests can be merged.
- The
master
/main
branch must "Require status checks to pass before merging" - The
master
/main
branch must "Require require branches to be up to date before merging"
So you're ready to start coding? If you haven't cloned this repository already you should, and then ...
- Clone this repository:
$ git clone git@github.com:HackYourFutureBelgium/javascript-30-starter.git
- Navigate to this repository in your local computer
$ cd javascript-30-starter
- Install the project's development dependencies - you will need these for the code quality automation:
$ npm install
This is a static website that only uses HTML, CSS and JavaScript, so you can run it using any static server. But the simplest way to get started is to use the dev server included with this repo:
npm run dev
: this will openindex.html
in your browser and automatically refresh whenever you change your code.
This project is a full website so you will need to run it by opening index.html
in the browser.
It's recommended that you use VSCode to write your HTML, CSS and JavaScript. Each time you make a change in VSCode you will need to go to the browser and refresh to see if it worked. Everything you have learned about the debugger will come in handy! You can use breakpoints and debugger
statements to step through while your website is running.
The only code in this project that you can develop and test separately from the rest is functions in the /src/logic
folder, these are pure functions that will need to test. You can develop these either in VSCode or Study Lenses.
Writing code is hard. To write even just 10 lines there are 100 things you need to think about, and 1000 mistakes you can make. Developers are clever and lazy people. They have built tools to help with all of this.
This repository comes with features to help you and your group write good code. Some features are for checking your code manually before pushing it, and others are for checking your code automatically when you open a PR to the main
/master
branch.
checks on GitHub
Remember the .lint-pass.txt
, .test-pass.txt
, ... files from Practice Code Review? Enough of that! It's time for the real deal. With this project repository your code quality results will not be written into separate files, but will be checked automatically by GitHub. Any mistakes and your PR not be able to merge.
When you open a PR to main
/master
in your repository, GitHub will run the workflows in ./.github/workflows:
- Lint JavaScript CI: Lints all the .js files in your
/src
folder, if there are any warnings or errors the check will fail. - Lint CSS CI: lints all the .css files in your
/public
folder, if there are any warnings or errors the check will fail. - Test CI: runs all the
.spec.js
fils in your/src
folder. If any tests fail, this check will fail. - Document CI: re-renders the
DOCS.md
file in the root of your repository, making sure that the documentation is always up to date. If the documentation script has any trouble the check will fail.
You will not be able to merge your branch until all the checks pass. These checks will be run again each time you push changes to your branch, so no worries if you don't pass the checks on your first try.
checks on your own computer
You can run the same checks locally to make sure you have no errors before pushing, this will make everything easier for you and your reviewer. If the checks pass locally then they should pass when you push.
- Linting JavaScript
npm run lint:js -- ./path/to/file.js
: lint one file at a time by passing the path to that filenpm run lint:js:ci
: lint all of the JavaScript in/src
. This is the same command that is run in GitHub so if it works locally it should pass the checks when you push.
- Linting CSS
npm run lint:css -- ./path/to/file.css
: lint one file at a time by passing the path to that filenpm run lint:css:ci
: lint all of the CSS in/public
. This is the same command that is run in GitHub so if it works locally it should pass the checks when you push.
- Testing
npm run test -- ./path/to/file.spec.js
: test one.spec.js
file at a timenpm run test:ci
: test all.spec.js
files at once. This is the same command that is run in GitHub so if it works locally it should pass the checks when you push.
- Formatting
npm run format -- ./path/to/file
: you can format any file type with this command (.js, .css, .html, .md)- if you install the Prettier extension for VSCode and have it format on save, then you don't need to use this command
- there is no extra CI check for formatting, but the linter will check some formatting. The rest is up to your code reviewer
- Documenting
npm run document
: render the documentation before pushing to make sure there are no errors.