Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci: add github actions workflow #303

Merged
merged 1 commit into from
Jan 15, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,10 @@ Please also list any relevant details for your test configuration
- [ ] Test B

**Test Configuration**:
* OS + version: e.g. macOS Mojave
* NPM version: ...
* Node version: ...

- OS + version: e.g. macOS Mojave
- NPM version: ...
- Node version: ...

## Checklist:

Expand All @@ -37,6 +38,5 @@ Please also list any relevant details for your test configuration
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream modules
- [ ] I have checked my code and corrected any misspellings
41 changes: 41 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
name: build
on: [push]
jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
# waiting on: https://github.com/actions/setup-node/issues/531
- run: corepack enable
- uses: actions/setup-node@v4
with:
node-version: 21
cache: npm
- run: npm ci

- name: test build package on node@21 (current)
run: |
node --version
npm --version
npm run test

# Not using a matrix here since it's simpler
# to just duplicate it and not spawn new instances

- uses: actions/setup-node@v4
with:
node-version: 20
- name: test build package on node@20 (LTS)
run: |
node --version
npm --version
npm run test

- uses: actions/setup-node@v4
with:
node-version: 18
- name: test build package on node@18 (LTS)
run: |
node --version
npm --version
npm run test
7 changes: 6 additions & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -1,12 +1,17 @@
# Contributing to `esLint-plugin-tailwindcss`

# Contributing
## Contributing

When contributing to this repository, please first discuss the change you wish to make via issue,
email, or any other method with the owners of this repository before making a change.

Please note we have a [code of conduct](CODE_OF_CONDUCT.md), please follow it in all your interactions with the project.

## Development

You can use [Corepack](https://nodejs.org/api/corepack.html) to ensure you're using the same package
manager. Run `corepack enabled` before running `npm install`.

## Pull Request Process

1. Ensure any install or build dependencies are removed before the end of the layer when doing a
Expand Down
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@
"typescript": "4.3.5",
"vue-eslint-parser": "^9.1.0"
},
"packageManager": "npm@10.2.5+sha256.8002e3e7305d2abd4016e1368af48d49b066c269079eeb10a56e7d6598acfdaa",
"engines": {
"node": ">=12.13.0"
},
Expand Down