Skip to content

Commit

Permalink
Merge branch 'release/2.1.0'
Browse files Browse the repository at this point in the history
  • Loading branch information
sequba committed Sep 8, 2022
2 parents e556026 + 7a2bf7f commit 25941f6
Show file tree
Hide file tree
Showing 234 changed files with 17,195 additions and 15,736 deletions.
2 changes: 1 addition & 1 deletion .config/source-license-header.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
/**
* @license
* Copyright (c) 2021 Handsoncode. All rights reserved.
* Copyright (c) 2022 Handsoncode. All rights reserved.
*/
18 changes: 12 additions & 6 deletions .config/webpack/development.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,18 +17,24 @@ module.exports.create = function create() {
c.devtool = 'source-map';
// Exclude all external dependencies from 'base' bundle
c.externals = {
moment: {
root: 'moment',
commonjs2: 'moment',
commonjs: 'moment',
amd: 'moment',
},
chevrotain: {
root: 'chevrotain',
commonjs2: 'chevrotain',
commonjs: 'chevrotain',
amd: 'chevrotain',
},
'tiny-emitter': {
root: 'TinyEmitter',
commonjs2: 'tiny-emitter',
commonjs: 'tiny-emitter',
amd: 'tiny-emitter',
},
unorm: {
root: 'unorm',
commonjs2: 'unorm',
commonjs: 'unorm',
amd: 'unorm',
},
};
c.plugins.push(new WebpackBar({ name: ` ${PACKAGE_FILENAME}.js` }));
});
Expand Down
9 changes: 1 addition & 8 deletions .config/webpack/production.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,14 +17,7 @@ module.exports.create = function create() {

c.optimization = {
minimize: true,
minimizer: [
new TerserPlugin({
terserOptions: {
mangle: false, // This option has to be disabled, otherwise GPU function doesn't work for min files.
compress: false, // This option has to be disabled, otherwise GPU function doesn't work for min files.
}
})
]
minimizer: [new TerserPlugin()]
};

c.plugins.forEach((plugin) => {
Expand Down
6 changes: 5 additions & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,11 @@ module.exports = {
'@typescript-eslint/no-unnecessary-boolean-literal-compare': ['error'],
'@typescript-eslint/no-extra-non-null-assertion': ['error'],
'@typescript-eslint/array-type': ['error'],
'@typescript-eslint/space-before-function-paren': ['error', 'never'],
'@typescript-eslint/space-before-function-paren': ["error", {
"anonymous": "never",
"named": "never",
"asyncArrow": "always"
}],
'@typescript-eslint/quotes': ['error', 'single', { avoidEscape: true }],
'@typescript-eslint/no-extra-semi': ['error'],
'@typescript-eslint/comma-spacing': ['error'],
Expand Down
29 changes: 16 additions & 13 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,15 +1,15 @@
### Context
<!--- Why is this change required? What problem does it solve? -->
<!--- Why are your changes required? What problem do they solve? -->

### How has this been tested?
<!--- Please describe in detail how you tested your changes (doesn't apply to translations and documentation changes). -->
### How did you test your changes?
<!--- Describe in detail how you tested your changes. -->

### Types of changes
<!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: -->
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] New feature or improvement (non-breaking change which adds functionality)
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] Additional language file or change to the existing one (translations)
<!--- What types of changes does your code introduce? Put an `x` in each box that applies. -->
- [ ] Breaking change (a fix or a feature because of which an existing functionality doesn't work as expected anymore)
- [ ] New feature or improvement (a non-breaking change that adds functionality)
- [ ] Bug fix (a non-breaking change that fixes an issue)
- [ ] Additional language file, or a change to an existing language file (translations)
- [ ] Change to the documentation

### Related issues:
Expand All @@ -18,9 +18,12 @@
3.

### Checklist:
<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
<!--- Go through the points below, and put an `x` in each box that applies. -->
<!--- If you're unsure about any of these, contact us. We're always glad to help! -->
- [ ] My change is compliant with the [OpenDocument](https://docs.oasis-open.org/office/OpenDocument/v1.3/os/part4-formula/OpenDocument-v1.3-os-part4-formula.html) standard.
- [ ] My change is compatible with Microsoft Excel.
- [ ] My change is compatible with Google Sheets.
- [ ] My code follows the code style of this project.
- [ ] I described the modification in the CHANGELOG.md file.
- [ ] My change requires a change to the documentation.
- [ ] My change requires a migration guide.
- [ ] I described my changes in the [CHANGELOG.md](https://github.com/handsontable/hyperformula/blob/master/CHANGELOG.md) file.
- [ ] My changes require a documentation update.
- [ ] My changes require a migration guide.
66 changes: 66 additions & 0 deletions .github/workflows/performance.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,66 @@
name: Performance

on:
pull_request:
types:
- opened
- reopened
- synchronize # the head branch is updated from the base branch, new commits are pushed to the head branch, or the base branch is changed

jobs:
performance-test:
strategy:
matrix:
node-version: [ '16' ]
os: [ 'ubuntu-latest' ]
name: performance-test
runs-on: ${{ matrix.os }}
steps:
- name: Setup Node.js ${{ matrix.node-version }}
uses: actions/setup-node@56899e050abffc08c2b3b61f3ec6a79a9dc3223d # https://github.com/actions/setup-node/releases/tag/v1.4.4
with:
node-version: ${{ matrix.node-version }}

- name: (base) Checkout
uses: actions/checkout@5a4ac9002d0be2fb38bd78e4b4dbde5606d7042f # https://github.com/actions/checkout/releases/tag/v2.3.4
with:
ref: ${{ github.event.pull_request.base.sha }}

- name: (base) Install dependencies
run: |
npm ci
- name: (base) Run performance tests
run: |
npm run benchmark:write-to-file base.json
- name: (head) Checkout
uses: actions/checkout@5a4ac9002d0be2fb38bd78e4b4dbde5606d7042f # https://github.com/actions/checkout/releases/tag/v2.3.4
with:
clean: false

- name: (head) Install dependencies
run: |
npm ci
- name: (head) Run performance tests
run: |
npm run benchmark:write-to-file head.json
- name: Compare the results
run: |
npm run benchmark:compare-benchmarks base.json head.json performance-report.md
- name: Publish a comment - header
uses: marocchino/sticky-pull-request-comment@6804b5ad49d19c10c9ae7cf5057352f7ff333f31 # https://github.com/marocchino/sticky-pull-request-comment/tree/v1.6.0
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
message: |
## Performance comparison of head (${{ github.event.pull_request.head.sha }}) vs base (${{ github.event.pull_request.base.sha }})
- name: Publish a comment - performance comparison report
uses: marocchino/sticky-pull-request-comment@6804b5ad49d19c10c9ae7cf5057352f7ff333f31 # https://github.com/marocchino/sticky-pull-request-comment/tree/v1.6.0
with:
append: true
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
path: performance-report.md
1 change: 0 additions & 1 deletion .npmignore
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,4 @@
!es/**/*
!typings/**/*
!github-hf-logo-blue.svg
!gpl-3.0.txt
!CHANGELOG.md
Loading

0 comments on commit 25941f6

Please sign in to comment.