Skip to content

Latest commit

 

History

History
164 lines (100 loc) · 6.64 KB

CONTRIBUTING.md

File metadata and controls

164 lines (100 loc) · 6.64 KB

Contributing to Yoshi

Hey! Thanks for your interest in improving our Toolkit!

Please take a moment to review this document in order to make the contribution process easy and effective for everyone involved.

Submitting an Issue

Please provide us with an issue in case you've found a bug, want a new feature, have an awesome idea, or there is something you want to discuss.

Submitting a Pull Request

Good pull requests, such as patches, improvements, and new features, are a fantastic help. They should remain focused in scope and avoid containing unrelated commits.

Please ask first if somebody else is already working on this or the core developers think your feature is in-scope. Generally always have a related issue with discussions for whatever you are including.

Local Setup

  1. Clone the repo git clone git@github.com:wix/yoshi.git.
  2. Run npm install in the yoshi directory.

That's it, you're good to go.

  • npm run test:templates - Create all create-yoshi-app's templates, install, build and test each one of them.
  • npm run test:integration - Create a few complex projects that cover a lot of different edge-cases, build, run, and test that everything is working.
  • npm run test:unit - Run the unit tests of all packages using jest.
  • npm run build - Run eslint on all packages with the following rules.

Adding a New Feature to the Yoshi Toolkit

  1. Make sure the feature is tested.
  2. Document it in README.md

Running test:integration Locally

Yoshi's test suite, in its current state, takes a long time to complete and (unfortunately) contains flaky tests. Therefore, we advise limiting the scope of the test execution in your local environment to the tests that are most affected by your changes. Limit the scope using mocha's only function.

After the limited scope of tests passes locally you can push your changes and have the Pull Request CI Server build and run all of the tests as the test suite is much less flaky on the CI server.

Test Phases

In order to simplify Yoshi's tests we created a helper utility called test-phases. This utility is in charge of setting up the environment for the test (package.json, pom.xml, source files, etc) in a temp directory, running Yoshi's commands (start, build, lint, etc) and asserting against the result (stdout, file content, exit code, etc). You can see an example usage of test-phases here.

Debugging Tests

You might run into an issue where you have a test that seems to run and then hang (neither fail nor pass). This usually means that there was an error but you can't see it. The reason behind this is that Yoshi mutes the output of all the tests by default in order not to spam the build log in the CI. In order to see the output of the tests, and see the error they threw, you can do one of the two:

it("should do something", () => {
  const res = test
    .verbose() // <------ add this
    .setup({
      //...
    })
    .execute(/* some task*/);

  expect(/* something */).to.equal(/* something */);
});

Alternatively, you can run all your tests (or just the focused ones) with the verbose flag in the following way:

VERBOSE_TESTS=true npm test

This is the same as adding the .verbose() method to each and every test.

Running test:templates

The E2E suite will create a corresponding E2E test for each template from projects/create-yoshi-app/templates directory. It will generate the project in a temporary directory, it will then run npm install & npm test to verify that it's not failing.

  • Verbose mode:

-v/--verbose output verbose logs, good for debugging

node ./scripts/runE2E.js --verbose
  • Focus specific templates using command lines arguments:

Example: will match client & fullstack-typescript projects

node ./scripts/runE2E.js client fullstack-typescript

Note: if no arguments supplied, the templates suite will run on all possible templates

Create Yoshi App

To work on the initializer, first link create-yoshi-app so it will be available globaly:

cd packages/create-yoshi-app
npm link

Adding a Template

Create two directories:

  • packages/create-yoshi-app/templates/<template-name>

  • packages/create-yoshi-app/templates/<template-name>-typescript

Specify them in the following file:

  • packages/create-yoshi-app/src/projects.js

It will be added to the templates you can choose when running create-yoshi-app and will be tested automatically in the e2e suite.

For details about the templating and more see create-yoshi-app

Modifying a Template

If you want to modify the content of one of the templates run the following command:

npm run create-yoshi-app:dev

It will create a project in a temp directory. You'll have a watcher that listens to changes from the template and updates the generated project accordingly.

Release a New Version

Start by heading to the CHANGELOG.md and insert the version's changes. For changes to appear in the changelog, the PRs should be tagged with tags like PR: New Feature :rocket:.

To autogenerate a changelog using last tagged PR's run:

npm run changelog

Note: For first time you'll need to export a GITHUB_AUTH token

  • New releases can be issued from branch master.

  • alpha/beta/rc versions should be issued from a branch named version_${version_name}.

To create a new version use the following command:

Note: Make sure you fetched up-to date tags from remote: git fetch --tags, lerna relies on tags for version information

npm run createVersion

This command will open an interactive UI for choosing the version, it will bump it in the relevant packages and add a git tag.

It runs lerna publish --skip-npm under the hood

Now Push the commits and tag to GitHub

git push origin master/version_* --follow-tags

In the end of the build (unless there is a failure) the release should be published to npm