-
Notifications
You must be signed in to change notification settings - Fork 59
Developer guide
This guide gathers good practices and elements that can help the developers to contribute to this project.
Each modification of the source code must respect our philosophy: consistency. A change to fix an issue or for a new feature must take into account the existing code and propagate the new rules or modification everywhere in the code.
This philosphy is applicable for technical concepts, but also for the rendering, wording and code formatting.
Templates exist for issues and pull requests. They must be used and filled out in details to help with the efficiency of the core team. Too much details is better than no details.
Please don't forget to check all the elements in the lists.
Continuous integration is put in place thanks to GitHub Actions. Most of these GitHub Actions are executable locally. The detailed commands are available in our package.json in the scripts section.
Based on the users' feedback, we are maintaining a detailed migration guide between versions. Each PR should be evaluated in terms of impact for the end-user:
- Change of HTML markup in components, documentation and examples
- New/deprecated features or components
- New/deprecated Sass/CSS variables
Most of the time, when new variables are added (CSS or Sass), it must be reflected in the documentation thanks to the <code>scss-docs</code> shortcode.