-
Notifications
You must be signed in to change notification settings - Fork 5
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #510 from Kipper-Lang/435-451-491-bug-fixes-sync-t…
- Loading branch information
Showing
76 changed files
with
4,085 additions
and
3,441 deletions.
There are no files selected for viewing
This file contains 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 |
|---|---|---|
| @@ -1,59 +1,83 @@ | ||
| <!-- | ||
| Please read through the given | ||
| Comments are marked by arrows, like in lines 1 and 5. They will not be visible in the final pull request! | ||
| --> | ||
|
|
||
| ## What type of change does this PR perform? | ||
|
|
||
| <!-- Add an x in the checkbox to mark it --> | ||
| <!-- Please put an X in the box of the line that applies --> | ||
| <!-- If you are unsure if your code is a breaking change, read this: https://nordicapis.com/what-are-breaking-changes-and-how-do-you-avoid-them --> | ||
|
|
||
| - [ ] Info or documentation change (Non-breaking change that updates dependencies, info files or online documentation) | ||
| - [ ] Website feature update or docs development changes (Change that changes the design or functionality of the websites or docs) | ||
| - [ ] Maintenance (Non-breaking change that updates dependencies) | ||
| - [ ] Info or documentation change (Non-breaking change that updates repo info files (e.g. README.md, CONTRIBUTING.md, etc.) or online documentation) | ||
| - [ ] Website feature update or docs development changes (Change that changes the design or functionality of the websites or docs) | ||
| - [ ] Development or internal changes (These changes do not add new features or fix bugs, but update the code in other ways) | ||
| - [ ] Bug fix (Non-breaking change which fixes an issue) | ||
| - [ ] New feature (Non-breaking change which adds functionality) | ||
| - [ ] Breaking change (Major bug fix or feature that would cause existing functionality to not work as expected.) | ||
| - [ ] Breaking change (Major bug fix or feature that would cause existing functionality not to work as expected.) | ||
| - [ ] Requires a documentation update, as it changes language or compiler behaviour | ||
|
|
||
| <!-- If you are unsure if your code is a breaking change, read this: https://nordicapis.com/what-are-breaking-changes-and-how-do-you-avoid-them --> | ||
|
|
||
| ## Summary | ||
|
|
||
| <!-- Explain the reason for this pr, changes and solution briefly. --> | ||
| <!-- Explain the reason for this pr, changes, and solution briefly. --> | ||
|
|
||
| <!-- Write here --> | ||
| <!-- REPLACE ME --> | ||
|
|
||
| Closes # <!-- Remove this if this is not related to an issue --> | ||
| <!-- Uncomment if this closes an issue: | ||
| Closes #INSERT_NR | ||
| --> | ||
|
|
||
| ## Summary of Changes | ||
| ## List of Changes | ||
|
|
||
| <!-- Please explain the changes in this PR and their influence. If this fixes an issue, describe what fixed the issue. --> | ||
|
|
||
| <!-- Write here --> | ||
| <!-- Create for every essential change a list item (Link any issues, discussions or PRs if needed!) --> | ||
|
|
||
| - Change Nr. 1 | ||
| - Change Nr. 2 | ||
| - ... | ||
|
|
||
| ## Does this PR create new warnings? | ||
|
|
||
| <!-- Add any new warnings or possible issues that could occur with this PR. --> | ||
|
|
||
| - Eslint reported ... | ||
| - CodeQL returned ... | ||
| - Warning Nr. 1 | ||
| - Warning Nr. 2 | ||
| - ... | ||
|
|
||
| <!-- Remove example text! --> | ||
| <!-- Just write none if they are no warnings, like this: | ||
| None. | ||
| --> | ||
|
|
||
| ## Detailed Changelog | ||
|
|
||
| _Not present for website/docs changes_ | ||
|
|
||
| <!-- Detailed changelog that may be copied from `CHANGELOG.md` (Only add the items you've added and remove any header with no item.). --> | ||
|
|
||
| <!-- ### Added --> | ||
| <!-- ### Changed --> | ||
| <!-- ### Removed --> | ||
| ### Added | ||
|
|
||
| ### Changed | ||
|
|
||
| ### Fixed | ||
|
|
||
| ### Deprecated | ||
|
|
||
| <!-- Default: --> | ||
| ### Removed | ||
|
|
||
| <!-- Just write none if they are no changelog entries (although you should definitely do some if they change source code), like this: | ||
| None. | ||
| --> | ||
|
|
||
| ## Linked issues or PRs | ||
|
|
||
| <!-- Include other issues and PRs related to this if any exist. Use this format: - [ ] #ISSUE_OR_PR --> | ||
|
|
||
| <!-- Default: --> | ||
| - [ ] #ISSUE_PR_OR_DISCUSSION_TAG | ||
| - [ ] #ISSUE_PR_OR_DISCUSSION_TAG | ||
| - [ ] ... | ||
|
|
||
| <!-- Just write the following if there are no linked issues: | ||
| No linked issues. | ||
| --> |
This file contains 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 |
|---|---|---|
|
|
@@ -25,7 +25,7 @@ jobs: | |
| node-version: ${{ matrix.node-version }} | ||
| - uses: pnpm/[email protected] | ||
| with: | ||
| version: 7.x.x | ||
| version: 8.x.x | ||
| run_install: false | ||
| - run: pnpm install --frozen-lockfile | ||
| - run: pnpm test | ||
|
|
||
This file contains 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 |
|---|---|---|
|
|
@@ -12,7 +12,7 @@ jobs: | |
|
|
||
| strategy: | ||
| matrix: | ||
| node-version: [14.x, 16.x, 18.x] | ||
| node-version: [16.x, 18.x] | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v3 | ||
|
|
@@ -22,7 +22,7 @@ jobs: | |
| node-version: ${{ matrix.node-version }} | ||
| - uses: pnpm/[email protected] | ||
| with: | ||
| version: 7.x.x | ||
| version: 8.x.x | ||
| run_install: false | ||
| - run: pnpm install --frozen-lockfile | ||
| - run: pnpm build | ||
|
|
@@ -34,7 +34,7 @@ jobs: | |
|
|
||
| strategy: | ||
| matrix: | ||
| node-version: [14.x, 16.x, 18.x] | ||
| node-version: [16.x, 18.x] | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v3 | ||
|
|
@@ -44,7 +44,7 @@ jobs: | |
| node-version: ${{ matrix.node-version }} | ||
| - uses: pnpm/[email protected] | ||
| with: | ||
| version: 7.x.x | ||
| version: 8.x.x | ||
| run_install: false | ||
| - run: pnpm install | ||
| - run: pnpm build | ||
|
|
@@ -56,7 +56,7 @@ jobs: | |
|
|
||
| strategy: | ||
| matrix: | ||
| node-version: [14.x, 16.x, 18.x] | ||
| node-version: [16.x, 18.x] | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v3 | ||
|
|
@@ -66,7 +66,7 @@ jobs: | |
| node-version: ${{ matrix.node-version }} | ||
| - uses: pnpm/[email protected] | ||
| with: | ||
| version: 7.x.x | ||
| version: 8.x.x | ||
| run_install: false | ||
| - run: pnpm install --frozen-lockfile | ||
| - run: pnpm build | ||
|
|
||
This file contains 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 |
|---|---|---|
|
|
@@ -9,7 +9,7 @@ jobs: | |
|
|
||
| strategy: | ||
| matrix: | ||
| node-version: [14.x, 16.x, 18.x] | ||
| node-version: [16.x, 18.x] | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v3 | ||
|
|
@@ -19,7 +19,7 @@ jobs: | |
| node-version: ${{ matrix.node-version }} | ||
| - uses: pnpm/[email protected] | ||
| with: | ||
| version: 7.x.x | ||
| version: 8.x.x | ||
| run_install: false | ||
| - run: pnpm install --frozen-lockfile | ||
| - run: pnpm test | ||
|
|
@@ -30,7 +30,7 @@ jobs: | |
|
|
||
| strategy: | ||
| matrix: | ||
| node-version: [14.x, 16.x, 18.x] | ||
| node-version: [16.x, 18.x] | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v3 | ||
|
|
@@ -40,7 +40,7 @@ jobs: | |
| node-version: ${{ matrix.node-version }} | ||
| - uses: pnpm/[email protected] | ||
| with: | ||
| version: 7.x.x | ||
| version: 8.x.x | ||
| run_install: false | ||
| - run: pnpm i | ||
| - run: pnpm test | ||
This file contains 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 |
|---|---|---|
| @@ -1,22 +1,30 @@ | ||
| name: "Size Limit" | ||
| on: pull_request | ||
| jobs: | ||
| size: | ||
| size-limit: | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| matrix: | ||
| node-version: [16.x] | ||
| env: | ||
| CI_JOB_NUMBER: 1 | ||
| steps: | ||
| - uses: actions/checkout@v3 | ||
| - uses: pnpm/[email protected] | ||
| - name: Checkout repository | ||
| uses: actions/checkout@v3 | ||
| - name: Use Node.js ${{ matrix.node-version }} | ||
| uses: actions/setup-node@v3 | ||
| with: | ||
| version: 7.0.0 | ||
| node-version: ${{ matrix.node-version }} | ||
| - name: Install PNPM & dev dependencies | ||
| uses: pnpm/[email protected] | ||
| with: | ||
| version: 8.x.x | ||
| run_install: | | ||
| - recursive: true | ||
| - name: Build project | ||
| - name: Build project to allow for browserify to run | ||
| run: pnpm build | ||
| - name: Setup size-limit | ||
| run: pnpm i | ||
| - uses: andresz1/[email protected] | ||
| - name: Run size-limit | ||
| uses: andresz1/[email protected] | ||
| with: | ||
| github_token: ${{ secrets.GITHUB_TOKEN }} | ||
| script: pnpm exec size-limit --json | ||
|
|
||
This file contains 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 |
|---|---|---|
|
|
@@ -15,3 +15,4 @@ kipper-standalone.min.js | |
| kipper/web/src/ | ||
| *.yaml | ||
| *.json | ||
| CITATION.cff | ||
This file contains 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.