Skip to content

Latest commit

 

History

History
151 lines (96 loc) · 6.06 KB

CONTRIBUTING.md

File metadata and controls

151 lines (96 loc) · 6.06 KB
Raw

Contributing

Introduction

First, thank you for considering contributing to Ts.ED! It is people like you that make the open source community such a great community! 😊

We welcome any type of contribution, not just code. You can help with:

  • QA: file bug reports, the more details you can give the better (e.g. screenshots with the console open)
  • Marketing: writing blog posts, how to's, printing stickers, ...
  • Community: presenting the project at meetups, organizing a dedicated meetup for the local community, ...
  • Code: take a look at the open issues. Even if you can't write code, commenting on them and showing that you care about a given issue matters. It helps us triage them.
  • Money: we welcome financial contributions in full transparency on our open collective.

Your First Contribution

Working on your first Pull Request? You can learn how from this free series: How to Contribute to an Open Source Project on GitHub.

Submitting code

Any code change should be submitted as a pull request. The description should explain what the code does and give steps to execute it. The pull request should also contain tests. Code review process

The bigger the pull request, the longer it will take to review and merge. Try to break down large pull requests in smaller chunks that are easier to review and merge. It is also always helpful to have some context for your pull request. What was the purpose? Why does it matter to you?

WARNING

Ts.ED project uses conventional commits as format commit message.

Release note and tagging version are based on the message commits. If you don't follow the format, our CI won't be able to increment the version correctly and your feature won't be released on NPM.

To write your commit message, see convention page here

Financial contributions

We also welcome financial contributions in full transparency on our open collective. Anyone can file an expense. If the expense makes sense for the development of the community, it will be "merged" in the ledger of our open collective by the core contributors, and the person who filed the expense will be reimbursed.

Questions

If you have any questions, create an issue (protip: do a quick search first to see if someone else didn't ask the same question before!). You can also reach us at https://gitter.im/Tsed-io/community.

How to work on Ts.ED

Setup

Clone your fork of the repository

$ git clone https://github.com/YOUR_USERNAME/tsed-formio.git

Install npm dependencies with yarn (not with NPM!):

yarn

After installing dependencies, yarn/npm run the postinstall hook and mounted all packages with npm link.

Compile TypeScript:

yarn build

Test

yarn test

Start a feature branch

git fetch
git branch --no-track -b feat-branch-name origin/master  # !IMPORTANT
yarn

## OR
gflow new feat name_of_feat

Commit & Push a feature

This command rebases your branch feature from the master branch, runs the test, and pushes your branch.

git commit -m "feat(domain): Your message"

Then:

npm run test
git fetch
git rebase origin/master
git push -f

# OR using gflow (run fetch, rebase and push for you)
gflow push

When your feature is ready to review, you can open a PR on Ts.ED github.

Submitting your PR

Create the PR on the master branch. A valid PR must follows these points:

  • Unit test is correctly implemented and cover the new scenario.
  • If the code introduce new feature, please add documentation in the /docs or describe the feature in the PR description.
  • The commit message follows the conventional commits format.

The review is always takes priority over other tasks. We can validate the PR very quickly if everything is clear and it's aligned with the evolution/vision of the framework.

Guidelines

  • All Ts.ED projects follows the git flow to generate a release note. To write your commit message see convention page.
  • Please try to combine multiple commits before pushing.
  • Please use TDD when fixing bugs. This means that you should write a unit test that fails because it reproduces the issue, then fix the issue and finally run the test to ensure that the issue has been resolved. This helps us prevent fixed bugs from happening again in the future.
  • Please keep the test coverage. Write additional unit tests if necessary.
  • Please create an issue before sending a PR if it is going to change the public interface of Ts.ED or includes significant architecture changes.
  • Feel free to ask for help from other members of the Ts.ED team.

Credits

Contributors

Backers

Thank you to all our backers! 🙏 [Become a backer]

Sponsors

Support this project by becoming a sponsor. Your logo will show up here with a link to your website. [Become a sponsor]