Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update CI to test Node16 #2987

Merged
merged 21 commits into from
May 13, 2022
Merged

Update CI to test Node16 #2987

merged 21 commits into from
May 13, 2022

Conversation

calebmshafer
Copy link
Member

@calebmshafer calebmshafer commented Jan 5, 2022 •
edited
Loading

Update the CI builds to run Node 16 by default, update documentation references to Node 14 to Node 16, and update the @types/node version to Node 16 version.

@calebmshafer
Copy link
Member Author

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Jan 20, 2022

update

✅ Branch has been successfully updated

@calebmshafer calebmshafer requested review from a team as code owners January 21, 2022 00:28
@calebmshafer calebmshafer mentioned this pull request Jan 25, 2022
Merged
@calebmshafer
Copy link
Member Author

@aruniverse the problem ended up being with the pretest script because nyc was launching the whole npm run test as a separate process which caused the pretest copy with cpx was the part that failed. I assume because it wasn't getting the correct working directory that it needed, but I didn't confirm that suspicion.

FYI @MichaelBelousov I did change this around a bit. The copy test assets is already done during the build test and not in the test step anymore. If you don't want to do that let me know but we'll need to figure out a workaround to the issue I called out above.

@calebmshafer calebmshafer merged commit 64c8d99 into master May 13, 2022
@calebmshafer calebmshafer deleted the node-16 branch May 13, 2022 01:47
pmconne added a commit that referenced this pull request Jun 21, 2022
@pmconne @calebmshafer
@aruniverse @mdastous-bentley @imodeljs-admin @MichaelBelousov @johnnyd710 @GerardasB @bbastings @saskliutas @grigasp @Alina657 @bsteinbk @ekandy @tcobbs-bentley @skirby1996 @DStradley @rschili @anmolshres98 @markschlosseratbentley @GytisCepk @veekeys @wgoehrig @nick4598 @paulius-valiunas @christophermlawson @jzbuchalski @DanielToby @kabentley @NancyMcCallB @MartynasStrazdas @raplemie @roluk @dassaf4 @mergify @kckst8 @AlainRobertAtBentley @naveedkhan8067
Merge @bentley/imodeljs-native 3.3.3 into imodel02 (#3836)
576d041 
* Update changelog for 3.2.0 (#3613)

* grouping, TOC

* TOC, verbosity

* Add missing features.

* Update CI to test Node16 (#2987)

* Update CI to test Node16

* update doc references to Node 14
* update types/node version

* cleanup map-layers-auth
* Improved documentation in README.md

* get transformer cover to pass on windows, need to investigate more
* remove pretest script from transformer

Co-authored-by: Arun George <[email protected]>
Co-authored-by: Michel D'Astous <[email protected]>

* 3.2.0-dev.73

* Multi-way viewport sync (#3614)

* multi-viewport sync.

* multi sync

* wip tests.

* wip more tests.

* finish tests.

* rename files.

* docs

* extract-api

* @extensions

* lint

* NextVersion

* clearer parameter name

* typo

* dta keyins can operate on more than 2 viewports.

* README

* inaccurate doc:

* 3.2.0-dev.74

* 3.3.0-dev.0

* close transformer state dumps even if overrides threw an error (#3612)

* close transformer state dumps even if producing them threw an error

Co-authored-by: Michael Belousov <[email protected]>

* 3.3.0-dev.1

* 3.3.0-dev.2

* 3.3.0-dev.3

* Extensions: Avoid Webpack critical dependency warning using a FunctionConstructor (#3616)

* Avoid webpack critical dependency warning using FunctionConstructor

* rush change

* rush change (with changelog entry)

Co-authored-by: Caleb Shafer <[email protected]>

* Fix a display issue with background color's ColorPicker where part of the dialog was missing. (#3617)

* Set default color input type to rgb on ColorPickerDialog

* Change log

* 3.3.0-dev.4

* update extension codeowners (#3630)

Co-authored-by: Arun George <[email protected]>

* Correctly remove a widget in FLOATING_WIDGET_SEND_BACK (#3619)

* Remove a widget in FLOATING_WIDGET_SEND_BACK.

* Rush change.

* Typo.

* [Extension Api]: Stable auto generated api (#3627)

* try to clean up and sort the generation script

Co-authored-by: Arun George <[email protected]>
Co-authored-by: Paul Connelly <[email protected]>

* Catch snap abandoned exception. (#3634)

* 3.3.0-dev.5

* Presentation: Fix hierarchy auto update (#3628)

* Set correct children count after update

* rush change

* extract-api

* Presentation: Fix excluding related properties without values (#3639)

* 3.3.0-dev.6

* Add learning snippets for property specification isReadOnly and priority attributes (#3640)

Co-authored-by: Alina Paliulionyte <[email protected]>

* Fix sizing of color picker dialog (#3623)

* Remove hardcoded height from color dialog.

* rush change

* UI: TypeConverter improvements (#3589)

* Return default number value instead of NaN

* rush change

* Return undefined instead of NaN

* rush change

* add isolate elements feature to transformer test app (#3601)

* add isolateElements tool to the transformer test-app

* add geometry options

* add to README note about the new isolatedElements feature

* fix cloneUsingJsonGeometry option description

Co-authored-by: Michael Belousov <[email protected]>

* Prevent duplicate logo cards. (#3648)

* 3.3.0-dev.7

* Remove cached fronstageDef went FrontstageProvider with same id is re-registered (#3647)

* Remove cached frontstageDef if frontstageProvider is re-registered.

* Add test and change log.

* 3.2.0 changelogs (#3654)

* 3.2.0

* rush change

Co-authored-by: imodeljs-admin <[email protected]>
Co-authored-by: Arun George <[email protected]>

* Compression branch (#3658)

* added compression

* added changelog

* Omit auth in RPC communication between mobile frontend and backend (#3649)

* Omit access token when mobile frontend talks to backend

* Changelog entry for mobile RPC auth omit

* Do not add backport reviewers if they have already approved the PR (#3659)

* Make display-test-app env vars use current values (#3644)

* change for env vars

* added shutdown to FrontEndDevTools and EditTool

* Fix drag target processing and update panel background color (#3635)

* Update to only confine floating widget to ninezone area on drag end and get proper widget/panel-section id when docking.

* Fix line height of overflow widget tabs when only icons are shown.

* Updates to avoid svg-loader component from loading svg multiple times.

* update unit test

* update panel background to use same as buic-background-control so dispabled itwin ui components display properly

* rush change

* update buic-background-panel to match buic-background-dialog

* fix lint error

* extract api

* Convert negative ViewRect coordinates to zero. (#3653)

* Convert negative ViewRect coordinates to zero.

* silly C++ holdover.

* missing paren in doc.

* 3.3.0-dev.8

* Valgrind azure pipeline (#3602)

* valgrind yaml build configuration

* updated yaml for valgrind

* git config for rush requirement

* try different rush cover call for valgrind

* Update valgrind.yaml for Azure Pipelines

* Update valgrind.yaml for Azure Pipelines

* Update valgrind.yaml for Azure Pipelines

* Update valgrind.yaml for Azure Pipelines

* Update valgrind.yaml for Azure Pipelines

* add suppression file for valgrind

* updated path to suppressions file

* switch to hosted agent

* switch to apt-get

* Increase timeout for valgrind tests

* Update valgrind.yaml for Azure Pipelines

* Update valgrind.yaml for Azure Pipelines

* Update valgrind.yaml for Azure Pipelines

* 3.3.0-dev.9

* 3.3.0-dev.10

* Upgrade mocha to latest version (10.0.0) (#3651)

* set mocha version for all projects to 10.0.0

Co-authored-by: Anmol Shrestha <[email protected]>

* 3.3.0-dev.11

* Fix background map base color transparency mix problem (#3668)

* Reality mesh shader needs to mix base color with any overrides even if textures are not specified (map base color, for example, can be specified).

* rush change

* Fix schedule timeline by clamping current duration to specified range - closes issue #75 (#3673)

* Ensure current duration does not go out of range.

* rush change

* [Extension Service Provider] add option to override access token (#3665)

* add option to override access token to support dual tokens
Co-authored-by: Arun George <[email protected]>

* move valgrind runs to be nightly instead of every pr (#3676)

Co-authored-by: Arun George <[email protected]>

* UI: Fix performance of getting subject models (#3666)

* 3.3.0-dev.12

* Do not create a Batch with an empty FeatureTable (#3684)

* test degenerate facets.

* Fix.

* remove spurious type assertion.

* mac build agents are slow.

* increase default timeout

* 3.3.0-dev.13

* Fix assertion on valid case in IModelExporter.exportElement. (#3688)

* 3.3.0-dev.14

* Close popup when the widget tab is clicked (#3685)

* Allow tippy.js to close the popup when the tab is clicked.

* Prevent browser drag interaction when dragging a tab.

* Allow tippy.js to close the popup when the widget is resized.

* Rush change.

* Tests.

* 3.3.0-dev.15

* 3.3.0-dev.16

* 3.3.0-dev.17

* 3.3.0-dev.18

* Add workarounds for Mali-G72 GPU. (#3701)

Co-authored-by: Arun George <[email protected]>

* 3.3.0-dev.19

* Presentation: Update IModelContent ruleset (#3698)

* Update the hierarchy

* Fix geometric elements under geometric models not being loaded

* UI: Clean up unused code in Models Tree (#3697)

* Update itwinui-react to 1.38.1 (#3694)

* Update valgrind.yaml for Azure Pipelines (#3683)

* Fix initial pending request always being treated as an error. (#3660)

* Add caching to some RPC Operations (#3594)

* Initial commit

* can't use break when not in a loop oops

* rush change

* add test

* 23 hours for getTileCachecONtainerUrl

* fix ocmment

* Use the decorator for responsecaching in a test

* bump tile and read rpc interface versions

* approach cache-control header in a different way

* extrac-tapi

* extract-api

* remove @todo and just keep it as TODO

* remove .only

* Revert "approach cache-control header in a different way"

This reverts commit 9246de9.

* extract-api

Co-authored-by: Caleb Shafer <[email protected]>

* Fix selection scope overrides (#3704)

* Ensure SelectionScope honor label overrides.

* rush change

* 3.3.0-dev.20

* Add OpenTelemetry tracing (#3548)

* add opentelemetry tracing

* rush update+change+extract

* fix dependencies

* remove unused import

* remove @opentelemetry/api dependency

* extract-api

* Refactor object flattening for OT span attributes.

* minor fixes

* WIP: Move OT support out of Logger.

* stop using deprecated types

* extract-api

* extract-api again

Co-authored-by: Bill Goehrig <[email protected]>

* 3.3.0-dev.21

* 3.3.0-dev.22

* 3.3.0-dev.23

* 3.3.0-dev.24

* Add SchemaContext.getKnownSchemas method (#3734)

* Add SchemaContext.getKnkownSchemas method

* change log

* api extraction

* test fix

* api extract

* 3.3.0-dev.25

* 3.2.1 changelogs (#3757)

Co-authored-by: imodeljs-admin <[email protected]>

* Ignore shader uniform errors by default (#3754)

* Ignore shader uniform errors by default

* Ignore shader uniform errors by default

* extract-api

* extract-api again

* Change to public

* enable logger for dta, doc new env

* Provide getter for active state (#3664)

* Provide getter for active state

Co-authored-by: Caleb Shafer <[email protected]>

* 3.3.0-dev.26

* Remove Android / iOS Distinction in core-mobile. Remove AuthStatus from BentleyError and StatusCategory. (#3622)

* Remove Android / iOS Distinction in core-mobile. Remove AuthStatus from BentleyError and StatusCategory.

* UI: Fix `useTreeModel` returning stale model for the given model source (#3761)

* 3.3.0-dev.27

* Move LocalHub to core-backend (#3763)

* move LocalHub to @internal in core-backend

* rush change

* Add addCustomAttribute to EC Class and Property method to editing API (#3744)

* Add ability to add CA to Property in editing API

* wip

* change log

* remove it.only

* PR review updates

* PR updates

* fix lint errors

* Presentation: Add support for nth level element selection scopes (#3767)

* Move HubMock to core-backend (#3770)

* move HubMock to core-backend

* rush change

* 3.3.0-dev.28

* Presentation: Add a way to get presentation backend's version by setting a flag in request params (#3756)

* 3.2.2 changelogs (#3799)

Co-authored-by: imodeljs-admin <[email protected]>

* [Extensions] ExtensionAdmin registerHost should accept hostnames and urls (#3631)

* ExtensionAdmin should accept hostnames and urls

* Remove the "_exists" method from the remote and extension service providers
* make iTwinId default to public on Service Extensions
* remove any cast and just log the error msg as is
* remove 'ftp' since its something we don't expect as input
* documentation fixes
* ServiceExtension props should not require a version - defaults to the latest
* attempt to parse the text response if body is null (happened during testing)

* 3.3.0-dev.29

* 3.3.0-dev.30

* 3.3.0-dev.31

* Change setImmediate to setTimeout (#3798)

* Lock cheerio version down (enzyme depedency) (#3800)

* Lock cheerio version down (enzyme depedency)

* lock down enzyme verion to 3.10.0

Co-authored-by: Caleb Shafer <[email protected]>

* Presentation: Custom nodes (#3772)

* Add renderer for nodes with too many children

* Render custom nodes

* Update pnpm-lock.yaml

* pnpm conflict fix

* extract-api fix

* extract-api fix 2

* Imports fix

* Imports sorted

* Add option for user nodeRenderer

* indentation fix

* indentation fix 2

* Naming changes

* rush update

Co-authored-by: Saulius Skliutas <[email protected]>

* export brep geometry when requested in IModelExporter (#3801)

* add roundtrip brep geom test

* export brep data through IModelExporter when wantGeometry is true

Co-authored-by: Michael Belousov <[email protected]>

* Create sphere and cylinder tools. Fixes for updateDynamics to supply adjusted point. (#3807)

* Create sphere and cylinder tools. Fixes for updateDynamics to supply ajusted point.

* Forgot to export new tool classes.

* 3.3.0-dev.32

* Clear drag state when button let up outside view. Don't send tools drag events if they didn't get button down event. (#3811)

* Fix tile display issues with ArcGIS map  layers (#3804)

* Make maplayers API public (#3745)

* Makes maplayers API public

* changelog

* Updated map-layers related API

* Promote more classes to public

* Added missing extract-api file.

* Expose MapLayerSource, MapLayerSourceProps, MapLayerSourceStatus, MapLayerSourceValidation

* Added MapLayerFormatId type.

* Replaced string enum with union type.

* Removed userName and password from MapLayerSourceProps

* Reverted formatId to string. Added MapLAyers group discussion. Use default parameters  in attachMapLayer and AttachMapLayerProps.

* Keep getMapLayerRange internal for now.

* Added change log.

* fix extract-api

* Use named parameters in DisplayTyleState.attachMapLayer and DisplayTyleState.attachMapLayerProps

* extract-api again

* 3.3.0-dev.33

* Use active tool name as tool settings label (#3814)

* Add useToolAsToolSettingsLabel option to UiFramework

* Extract api

* rush change

* 3.3.0-dev.34

* Presentation: Enable response compression for certain PresentationRpcInterface operations (#3818)

* 3.3.0-dev.35

* prevent Delaunay flipper from removing hole edges (#3731)

* Avoid slow keyin filtering when processing long keyins (#3830)

* Avoid doing slow filtering when not necessary

* revert change to filter

* rush change

* 3.3.0-dev.36

* how about we avoid running rush update on imodel02 branch.

* fix up core-backend

* fix CheckpointManager test.

* Delete core-transformer's HubMock

Temporarily comment out TileCache tests.

* unused imports.

* 3.3.0-dev.37

* fix TileCache.test.ts merge

* extract-api; doc TODOs

* lint

* invalid doc links.

* 3.3.0-dev.38

* Fix presentation-backend test

* Automated GitHub Releases (#3820)

* add python release script
* add release creation to version bump pipeline
* use gh action to create release

* remove unncessary dependencies from map-layers-auth

* re-remove merge-restored removed deepEqualWithFpTolerance impl

* Convert getModelProps RPC operation to a GET request from POST (#3835)

* change getModelProps to a GET request

* rush change

* Lock down ts-node to 10.8.0 for now because istanbuljs/nyc#1473

* fix slash direction

* Lock superagent down to 7.1.3 because 7.1.6 has core-full-stack-test integration test failures in electron.

* String.replaceAll no existe.

* Remove RPC preflight and InitializeInterface. (#3831)

* remove unnecessary pretest script that triggers an nyc bug

* 3.3.0-dev.39

* 3.3.0-dev.40

* fix test errors

* extract-api, lint

* remove temporary test setting for gcs

* fix lint error

Co-authored-by: Caleb Shafer <[email protected]>
Co-authored-by: Arun George <[email protected]>
Co-authored-by: Michel D'Astous <[email protected]>
Co-authored-by: imodeljs-admin <[email protected]>
Co-authored-by: Michael Belousov <[email protected]>
Co-authored-by: Michael Belousov <[email protected]>
Co-authored-by: John DiMatteo <[email protected]>
Co-authored-by: Arun George <[email protected]>
Co-authored-by: GerardasB <[email protected]>
Co-authored-by: bbastings <[email protected]>
Co-authored-by: Saulius Skliutas <[email protected]>
Co-authored-by: Grigas <[email protected]>
Co-authored-by: Alina Paliulionytė <[email protected]>
Co-authored-by: Alina Paliulionyte <[email protected]>
Co-authored-by: Bill Steinbock <[email protected]>
Co-authored-by: ekandy <[email protected]>
Co-authored-by: Travis Cobbs <[email protected]>
Co-authored-by: Seamus Kirby <[email protected]>
Co-authored-by: DStradley <[email protected]>
Co-authored-by: Robert Schili <[email protected]>
Co-authored-by: Anmol Shrestha <[email protected]>
Co-authored-by: Anmol Shrestha <[email protected]>
Co-authored-by: Mark Schlosser <[email protected]>
Co-authored-by: Gytis Čepkauskas <[email protected]>
Co-authored-by: Vykis <[email protected]>
Co-authored-by: Bill Goehrig <[email protected]>
Co-authored-by: nick4598 <[email protected]>
Co-authored-by: Paulius Valiūnas <[email protected]>
Co-authored-by: christophermlawson <[email protected]>
Co-authored-by: Joe Zbuchalski <[email protected]>
Co-authored-by: Daniel Toby <[email protected]>
Co-authored-by: kabentley <[email protected]>
Co-authored-by: NancyMcCallB <[email protected]>
Co-authored-by: Martynas <[email protected]>
Co-authored-by: Raphaël LEMIEUX <[email protected]>
Co-authored-by: Robert Lukasonok <[email protected]>
Co-authored-by: dassaf4 <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: kckst8 <[email protected]>
Co-authored-by: AlainRobertAtBentley <[email protected]>
Co-authored-by: Daniel Toby <[email protected]>
Co-authored-by: naveedkhan8067 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ColinKerr ColinKerr ColinKerr approved these changes

@pmconne pmconne pmconne approved these changes

@wgoehrig wgoehrig Awaiting requested review from wgoehrig wgoehrig is a code owner

@kabentley kabentley Awaiting requested review from kabentley kabentley is a code owner automatically assigned from iTwin/itwinjs-core

@bbastings bbastings Awaiting requested review from bbastings bbastings is a code owner

@bsteinbk bsteinbk Awaiting requested review from bsteinbk

@dassaf4 dassaf4 Awaiting requested review from dassaf4 dassaf4 is a code owner

@mdastous-bentley mdastous-bentley Awaiting requested review from mdastous-bentley mdastous-bentley is a code owner

@MarcBedard8 MarcBedard8 Awaiting requested review from MarcBedard8

@williamkbentley williamkbentley Awaiting requested review from williamkbentley

@aruniverse aruniverse Awaiting requested review from aruniverse aruniverse is a code owner

@Ellord207 Ellord207 Awaiting requested review from Ellord207

@jchick-bentley jchick-bentley Awaiting requested review from jchick-bentley

@diegoalexdiaz diegoalexdiaz Awaiting requested review from diegoalexdiaz

@MichaelBelousov MichaelBelousov Awaiting requested review from MichaelBelousov

@mgooding mgooding Awaiting requested review from mgooding

@paulius-valiunas paulius-valiunas Awaiting requested review from paulius-valiunas

@ramanujam-raman-zz ramanujam-raman-zz Awaiting requested review from ramanujam-raman-zz

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@calebmshafer @pmconne @ColinKerr @aruniverse @mdastous-bentley