●Contributing
We're honored that people want to contribute to our little project.
Let's build a public good together :)
Our priorities now fall in 3 buckets:
- bugs: mailing should be relatively bug-free
- compatibility: mailing should work in many environments
- features: mailing should be fun and powerful for developing and sending emails
The first 2 categories are straightforward, go ahead and grab an open issue by assiging yourself or calling dibs with a comment. Even submitting a cold PR with a bugfix is welcome.
The third category is trickier. Adding features can complicate existing use-cases or make mailing more difficult to use so we want to be careful. Therefore, let's discuss new features in discussions or issues before working on them. If you want to hack together a prototype to flesh out your idea before discussing that's cool too, just know that it's not likely to be merged as-is.
Dev Setup
git clone git@github.com:sofn-xyz/mailing.git
cd mailing
yarn
yarn dev
yarn dev
starts the cli in dev mode
Develop using a demo next app
For development, you may want to have a demo next app that pulls in your changes. We've had success using yalc[https://github.com/wclr/yalc] and the following flow:
- Register
mailing
as a local package withyalc
: in thepackages/cli
directory, runyalc add
. - Create a new next app in your projects directory by running
yarn create next-app --typescript
for a typescript app ORyarn create next-app
for a js app - In the next app, run
yalc add mailing
, this createsnode_modules/mailing
andnode_modules/.bin/mailing
. (Note:yarn link
does not add the bin file, which is whyyalc
is prefered) - Make your changes in
mailing
- Run
yarn build
in themailing
root directory to create newdist
files - Run
yalc push
in themailing
root directory to both publish your changes (yalc publish
) and pull them in to your next app (yalc update
)
Run cypress tests for the preview server
- Start a mailing preview server on localhost:3883
- cd into packages/cli and run
yarn cypress run
Run embedded jest tests for the preview server
During the framework test process described below, the jest tests in scripts/e2e_test/jest_tests
are copied into the directory where each target framework is installed and run. Before testing them in the framework install context, however, you will want to make sure they pass on the latest build by running yarn build
and then yarn e2e:jest
in the mailing project root.
The directory scripts/e2e_test
contains framework tests targeting supported frameworks that should be run before every public npm release. Each test uses the yarn create
command to create new projects from their create-*
starter kits and then runs the cypress cli tests contained in packages/cli/cypress
and the jest tests contained in scripts/e2e_test/jest_tests
.
The frameworks currently covered by the tests are:
- standalone (no framework)
- turbo (Turbo monorepo)
- next_ts (Next.js with Typescript)
- next_js (Next.js without Typescript)
- redwood_ts (Redwood with Typescript)
- redwood_js (Redwood without Typescript)
- remix_ts (Remix with Typescript)
- remix_js (Remix without Typescript)
Initial test setup
- In the project root, run
asdf install
to install ruby 3.1.2 - In the directory
scripts/e2e_test
, runbundle install
to install the required ruby gems
Run the framework tests
- In the project root, run
yarn test:e2e
to run the full framework test suite, including cypress and jest tests for all supported frameworks.
This will instantiate each framework, add mailing with yalc, and then run the cypress tests contained in packages/cli/cypress
and the jest tests contained in scripts/e2e_test/jest_tests
.
Run the framework tests with advanced options
The underlying ruby script bundle exec ruby e2e/cli.rb
supports some options for running:
--app=redwood_ts
to run the tests only on the specified framework. Seee2e/app.rb
for the list of supported frameworks--skip-build
to skip the yarn build part of the script, useful when debugging something unrelated to the build--update-snapshot
if you need to update the snapshots in the framework tests. This will run jest with the-u
option and then copy--rerun
to skip the framework install part of the script, useful when debugging something in your cypress tests unrelated to the build or the framework install. This will use the framework installs that are present in the runs/latest directory, i.e. the assumption is you've run a test against some framework(s) and you now want to re-running them after adjusting your cypress tests. the updated snapshots back to mailing.
Cache the framework installs for faster runs
- Use the
--save-cache
flag to save each framework install (before mailing is added) to thecache
directory. Subsequent test runs will start with a copy of the cache instead of runningyarn create
andyarn install
, which will speed things up 🏎 If you need to resetthe cache, e.g. if you want to test a newer version of the framework or if the framework install process changes, you can delete the cache directory or the subdirectory containing the specific framework you are targeting.