Skip to content

Commit

Permalink
Merge pull request #107 from Expensify/monil-addTestingSteps
Browse files Browse the repository at this point in the history
Add testing steps to the README file
  • Loading branch information
carlosmiceli authored Jul 31, 2024
2 parents 99e1dd3 + ab41818 commit 25372fb
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,14 @@ This package provides Expensify's .eslintrc as an extensible shared config. Most
1. Make a change
2. Bump the version in package.json and submit your PR

### Testing

After you have submitted a PR with an updated version in the `package.json` file,

1. Get the full commitID of the last commit in your PR, and run `npm install git+https://github.com/Expensify/eslint-config-expensify.git#COMMIT_ID` in the repo against which you want to test those changes.
2. This should update the version of `eslint-config-expensify` in `package-lock.json` file, and ensures the repo is referencing to the correct version of the eslint config.
3. Now, you can run `npm run lint` or perform any other tests you want in that repo.

### After PR is merged
You need to publish the newest version of this to NPM so that we can update it in the other repos
1. From the command line, in the directory for this repo:
Expand Down

0 comments on commit 25372fb

Please sign in to comment.