Merge pull request #1052 from cewert/update-workflow-dependencies
Fix npm security vulnerabilities
This commit is contained in:
1hitsong
GitHub
commit
9fed8a61d2
25
.github/ISSUE_TEMPLATE/bug_report.md
vendored
25
.github/ISSUE_TEMPLATE/bug_report.md
vendored
|
|
@ -7,33 +7,34 @@ assignees: ''
|
|||
|
||||
---
|
||||
|
||||
**Software Versions**
|
||||
Jellyfin Server Version:
|
||||
Roku Client Version:
|
||||
## Software Versions
|
||||
|
||||
**Describe the bug**
|
||||
- Jellyfin Server Version:
|
||||
- Roku Client Version:
|
||||
|
||||
## Describe the bug
|
||||
<!-- A clear and concise description of what the bug is. -->
|
||||
|
||||
**How To Reproduce**
|
||||
## How To Reproduce
|
||||
<!-- Steps to reproduce the behavior: -->
|
||||
1. Go to '...'
|
||||
2. Click on '....'
|
||||
3. Scroll down to '....'
|
||||
4. Bug occurs
|
||||
|
||||
**Expected behavior**
|
||||
## Expected behavior
|
||||
<!-- A clear and concise description of what you expected to happen. -->
|
||||
|
||||
**Logs**
|
||||
## Logs
|
||||
<!-- Please paste any log errors. -->
|
||||
|
||||
**Screenshots**
|
||||
## Screenshots
|
||||
<!-- If applicable, add screenshots to help explain your problem. -->
|
||||
|
||||
**Connection Information**
|
||||
Is server local or remote?
|
||||
## Connection Information
|
||||
|
||||
Is server connection http or https?
|
||||
- Is server local or remote?
|
||||
- Is server connection HTTP or HTTPS?
|
||||
|
||||
**Additional context**
|
||||
## Additional context
|
||||
<!-- Add any other context about the problem here. -->
|
||||
|
|
|
|||
10
.github/ISSUE_TEMPLATE/enhancement-request.md
vendored
10
.github/ISSUE_TEMPLATE/enhancement-request.md
vendored
|
|
@ -1,20 +1,20 @@
|
|||
---
|
||||
name: Enhancement request
|
||||
about: Suggest an modification to an existing feature
|
||||
about: Suggest a modification to an existing feature
|
||||
title: ''
|
||||
labels: enhancement
|
||||
assignees: ''
|
||||
|
||||
---
|
||||
|
||||
**Is your feature request related to a problem? Please describe.**
|
||||
## Is your feature request related to a problem? Please describe
|
||||
<!-- A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] -->
|
||||
|
||||
**Describe the solution you'd like**
|
||||
## Describe the solution you'd like
|
||||
<!-- A clear and concise description of what you want to happen. -->
|
||||
|
||||
**Describe alternatives you've considered**
|
||||
## Describe alternatives you've considered
|
||||
<!-- A clear and concise description of any alternative solutions or features you've considered. -->
|
||||
|
||||
**Additional context**
|
||||
## Additional context
|
||||
<!-- Add any other context or screenshots about the feature request here. -->
|
||||
|
|
|
|||
4
.github/ISSUE_TEMPLATE/feature_request.md
vendored
4
.github/ISSUE_TEMPLATE/feature_request.md
vendored
|
|
@ -7,8 +7,8 @@ assignees: ''
|
|||
|
||||
---
|
||||
|
||||
**Describe the feature you'd like**
|
||||
## Describe the feature you'd like
|
||||
<!-- A clear and concise description of what you want to happen. -->
|
||||
|
||||
**Additional context**
|
||||
## Additional context
|
||||
<!-- Add any other context or screenshots about the feature request here. -->
|
||||
|
|
|
|||
6
.github/pull_request_template.md
vendored
6
.github/pull_request_template.md
vendored
|
|
@ -2,10 +2,10 @@
|
|||
Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y).
|
||||
For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our https://jellyfin.readthedocs.io/en/latest/developer-docs/contributing/ page.
|
||||
-->
|
||||
|
||||
**Changes**
|
||||
<!-- markdownlint-disable MD041 first-line-heading -->
|
||||
## Changes
|
||||
<!-- Describe your changes here in 1-5 sentences. -->
|
||||
|
||||
**Issues**
|
||||
## Issues
|
||||
<!-- Tag any issues that this PR solves here.
|
||||
ex. Fixes # -->
|
||||
|
|
|
|||
15
.github/workflows/lint.yml
vendored
15
.github/workflows/lint.yml
vendored
|
|
@ -58,3 +58,18 @@ jobs:
|
|||
- uses: xt0rted/markdownlint-problem-matcher@98d94724052d20ca2e06c091f202e4c66c3c59fb # v2
|
||||
- name: Lint markdown files
|
||||
run: npm run lint-markdown
|
||||
spelling:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: Clone github repo
|
||||
uses: actions/checkout@master
|
||||
- uses: actions/setup-node@master
|
||||
with:
|
||||
node-version: "lts/*"
|
||||
cache: "npm"
|
||||
- name: Install npm dependencies
|
||||
run: npm ci
|
||||
- name: Install roku package dependencies
|
||||
run: npx ropm install
|
||||
- name: Check markdown files for spelling errors
|
||||
run: npm run lint-spelling
|
||||
|
|
@ -1,5 +0,0 @@
|
|||
{
|
||||
"ignore": [
|
||||
"node_modules/**/*"
|
||||
]
|
||||
}
|
||||
17
.spelling
17
.spelling
|
|
@ -1,17 +0,0 @@
|
|||
jellyfin
|
||||
brightscript
|
||||
vscode
|
||||
roku
|
||||
github
|
||||
pre-release
|
||||
sideload
|
||||
dev
|
||||
repo
|
||||
hardcode
|
||||
hardcoding
|
||||
breakpoint
|
||||
DEVGUIDE
|
||||
runtime
|
||||
translations.ts
|
||||
en_US
|
||||
ing
|
||||
12
DEVGUIDE.md
12
DEVGUIDE.md
|
|
@ -60,16 +60,16 @@ We recommend using Visual Studio Code when working on this project. The [BrightS
|
|||
|
||||
### Usage
|
||||
|
||||
1. Open the `jellyfin-roku` folder in vscode
|
||||
2. Press `F5` on your keyboard or click `Run` -> `Start Debugging` from the vscode menu. 
|
||||
1. Open the `jellyfin-roku` folder in VSCode
|
||||
2. Press `F5` on your keyboard or click `Run` -> `Start Debugging` from the VSCode menu. 
|
||||
|
||||
3. Enter your Roku IP address and developer password when prompted
|
||||
|
||||
That's it! vscode will auto-package the project, sideload it to the specified device, and the channel is up and running. (assuming you remembered to put your device in [developer mode](#developer-mode))
|
||||
That's it! VSCode will auto-package the project, sideload it to the specified device, and the channel is up and running. (assuming you remembered to put your device in [developer mode](#developer-mode))
|
||||
|
||||
### Hardcoding Roku Information
|
||||
|
||||
Out of the box, the Brightscript extension will prompt you to pick a Roku device (from devices found on your local network) and enter a password on every launch. If you'd prefer to hardcode this information rather than entering it every time, you can set these values in your vscode user settings:
|
||||
Out of the box, the BrightScript extension will prompt you to pick a Roku device (from devices found on your local network) and enter a password on every launch. If you'd prefer to hardcode this information rather than entering it every time, you can set these values in your VSCode user settings:
|
||||
|
||||
```js
|
||||
{
|
||||
|
|
@ -95,7 +95,7 @@ Build the package
|
|||
make dev
|
||||
```
|
||||
|
||||
This will create a zip in `out/apps/Jellyfin_Roku-dev.zip`. Login to your roku's device in your browser and upload the zip file then run install.
|
||||
This will create a zip in `out/apps/Jellyfin_Roku-dev.zip`. Login to your Roku's device in your browser and upload the zip file then run install.
|
||||
|
||||
## Method 3: Direct load to Roku Device
|
||||
|
||||
|
|
@ -154,7 +154,7 @@ make install
|
|||
|
||||
Modify code -> `make install` -> Use Roku remote to test changes -> `telnet ${ROKU_DEV_TARGET} 8085` -> `CTRL + ]` -> `quit + ENTER`
|
||||
|
||||
Unfortunately there is no debugger. You will need to use telnet to see log statements, warnings, and error reports. You won't always need to telnet into your device but the workflow above is typical when you are new to Brightscript or are working on tricky code.
|
||||
Unfortunately there is no debugger. You will need to use telnet to see log statements, warnings, and error reports. You won't always need to telnet into your device but the workflow above is typical when you are new to BrightScript or are working on tricky code.
|
||||
|
||||
Install necessary packages:
|
||||
|
||||
|
|
|
|||
|
|
@ -30,7 +30,7 @@ The channel is available on the [Roku Channel Store](https://my.roku.com/add/jel
|
|||
|
||||
## Getting Involved<a name="get_involved"></a>
|
||||
|
||||
No matter what your interests or skill are, you can help to make this client better for everyone by simply using the client and letting us know if you find a problem with it. Either give us a shout on [matrix](https://matrix.to/#/+jellyfin:matrix.org) or create a github issue.
|
||||
No matter what your interests or skill are, you can help to make this client better for everyone by simply using the client and letting us know if you find a problem with it. Either give us a shout on [matrix](https://matrix.to/#/+jellyfin:matrix.org) or create a GitHub issue.
|
||||
|
||||
Feature requests are always welcome too, but please have a read though the existing issues to see if someone has already raised one for something similar.
|
||||
|
||||
|
|
|
|||
17
dictionary.txt
Normal file
17
dictionary.txt
Normal file
|
|
@ -0,0 +1,17 @@
|
|||
Jellyfin
|
||||
VSCode
|
||||
BrightScript
|
||||
sideload
|
||||
Sideload
|
||||
DEVGUIDE
|
||||
ing
|
||||
hardcode
|
||||
Hardcoding
|
||||
pre-release
|
||||
breakpoint
|
||||
repo
|
||||
Repo
|
||||
dev
|
||||
Dev
|
||||
assignees
|
||||
HTTPS
|
||||
6912
package-lock.json
generated
6912
package-lock.json
generated
File diff suppressed because it is too large
Load Diff
11
package.json
11
package.json
|
|
@ -7,17 +7,18 @@
|
|||
"@rokucommunity/bslint": "0.8.1",
|
||||
"brighterscript": "0.61.3",
|
||||
"ropm": "0.10.11",
|
||||
"jsonlint-cli": "1.0.1",
|
||||
"markdownlint-cli": "0.33.0",
|
||||
"markdown-spellcheck": "1.3.1"
|
||||
"jshint": "^2.13.6",
|
||||
"markdownlint-cli2": "0.6.0",
|
||||
"spellchecker-cli": "6.1.1"
|
||||
},
|
||||
"scripts": {
|
||||
"postinstall": "npx ropm copy",
|
||||
"validate": "npx bsc --copy-to-staging=false --create-package=false",
|
||||
"test": "echo \"Error: no test specified\" && exit 1",
|
||||
"lint": "bslint",
|
||||
"lint-json": "jsonlint-cli **/*.json",
|
||||
"lint-markdown": "markdownlint **/*.md --ignore node_modules && mdspell --en-us -r **/*.md !**/node_modules/**/*.md",
|
||||
"lint-json": "jshint --extra-ext .json --verbose --exclude node_modules ./",
|
||||
"lint-markdown": "markdownlint-cli2 \"**/*.md\" \"#node_modules\"",
|
||||
"lint-spelling": "spellchecker -d dictionary.txt --files \"**/*.md\" \"**/.*/**/*.md\" \"!node_modules/**/*.md\"",
|
||||
"check-formatting": "npx bsfmt --check",
|
||||
"format": "npx bsfmt --write"
|
||||
},
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user