-
Notifications
You must be signed in to change notification settings - Fork 89
Gitub actions #84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
submarcos
wants to merge
16
commits into
master
Choose a base branch
from
gitub_actions
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Gitub actions #84
Changes from all commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
e683795
backup old travis ci file
submarcos 24b3d2b
add github action workflow file
submarcos 961ca36
install peer dependencies in CI
submarcos 58d72e9
lint and publish in CI
submarcos 7f35e25
update badge to deal with github actions
submarcos 1fed508
fix lint CI job
submarcos b37e07a
use moved package in new mapbox repo
submarcos 39d543d
fix samples
submarcos f20477c
fix new package requirements
submarcos 26f2bf2
fix eslint
submarcos 02a13c9
fi eslint and lock file
submarcos 07bd264
fix dev view to use packaged scripts in dev
submarcos 0662687
Merge pull request #85 from makinacorpus/fix_eslint
submarcos f764670
delete travis ci file
submarcos caf5956
revert togeojson package change (other PR)
submarcos 06cdb91
fix eslint
submarcos File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,55 @@ | ||
| name: CI | ||
|
|
||
| on: | ||
| pull_request: | ||
| push: | ||
| branches: | ||
| - master | ||
| release: | ||
| types: | ||
| - created | ||
| schedule: | ||
| - cron: '0 2 * * *' | ||
|
|
||
| jobs: | ||
| lint: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - name: Setup node | ||
| uses: actions/setup-node@v2 | ||
| with: | ||
| node-version: 12 | ||
| cache: 'npm' | ||
| - run: npm ci | ||
| - run: npm run lint | ||
|
|
||
| test: | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| matrix: | ||
| node: [ '12', '14', '16' ] | ||
| name: Node ${{ matrix.node }} unit test | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - name: Setup node | ||
| uses: actions/setup-node@v2 | ||
| with: | ||
| node-version: ${{ matrix.node }} | ||
| cache: 'npm' | ||
| - run: npm ci | ||
| - run: npm install leaflet togeojson --no-save | ||
| - run: npm test | ||
|
|
||
| publish: | ||
| needs: [lint, test] | ||
| runs-on: ubuntu-latest | ||
| if: ${{ github.event_name == 'release' && github.event.action == 'created' }} | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - uses: actions/setup-node@v1 | ||
| with: | ||
| node-version: 12 | ||
| - uses: JS-DevTools/npm-publish@v1 | ||
| with: | ||
| token: ${{ secrets.NPM_TOKEN }} |
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
leaflet.css is still in 1.2.0, we should stay on the same version as the
.jsI can't comment on the line itself but there it is :
Leaflet.FileLayer/docs/index.html
Line 8 in 06cdb91