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.

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

Update integration typescript #2381

Open
wants to merge 117 commits into
base: integration/typescript
Choose a base branch
from

Conversation

djskinner
Copy link
Contributor

@djskinner djskinner commented Mar 18, 2025

merge latest next into integration/typescript. I've added comments covering the changes I've made to get this working (mostly with the new pipeline changes). The PR diff is failing I think due to the pipeline changes but can be seen working here: #2382

yousif-bugsnag and others added 30 commits February 17, 2025 11:17
* Move AWS lambda tests from BK to GHA

* Don't automatically bundle install at root

* Set node-version to download in the correct place

* Remove SAM check from maze env

* Test using localhost

* Use host depending on the running location

* Fix syntax

* Fix used steps to specific commits

* Debug test failures

* Revert to using released version of maze-runner
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.28.9 to 3.28.10.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@9e8d078...b56ba49)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [actions/cache](https://github.com/actions/cache) from 4.2.0 to 4.2.1.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](actions/cache@1bd1e32...0c907a7)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [ossf/scorecard-action](https://github.com/ossf/scorecard-action) from 2.4.0 to 2.4.1.
- [Release notes](https://github.com/ossf/scorecard-action/releases)
- [Changelog](https://github.com/ossf/scorecard-action/blob/main/RELEASE.md)
- [Commits](ossf/scorecard-action@62b2cac...f49aabe)

---
updated-dependencies:
- dependency-name: ossf/scorecard-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.6.0 to 4.6.1.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@65c4c4a...4cec3d8)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
* ci(react-native): add RN 0.78 tests

* test(react-native): bump react-native-screens version

* test(react-native): update version-dependent steps for 0.78
* Ensure correct heading name is used for changelog entry

* Re-enable full workflow and target bumpsnag branch
dependabot bot and others added 23 commits March 17, 2025 04:05
Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.224.0 to 1.226.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Changelog](https://github.com/ruby/setup-ruby/blob/master/release.rb)
- [Commits](ruby/setup-ruby@bbda858...922ebc4)

---
updated-dependencies:
- dependency-name: ruby/setup-ruby
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Add delay between layer starts in test scenario launcher
…nt-queue

Ensure detect changes script runs on node queue
Enable Appium session reporting in e2e tests
This reverts commit 80b1e18.
Comment on lines +35 to +37
"scripts": {
"build": "echo 'No build step required for @bugsnag/js'"
},
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

without this the build steps for dependencies weren't running first (e.g. @bugsnag/browser, plugins, core). Not really sure why this is an issue now.

@@ -28,7 +28,8 @@
"@angular-devkit/build-angular": "^19.0.5",
"@angular/cli": "^19.0.5",
"@angular/common": "^19.0.0",
"@angular/compiler-cli": "^19.0.0",
"@angular/compiler": "^19.2.2",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apparent missing dependency

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems to have to match @angular/compiler exactly

@@ -68,6 +68,7 @@
"@rollup/plugin-replace": "^6.0.2",
"@rollup/plugin-typescript": "^12.1.2",
"@types/jest": "^26",
"rollup": "^4.24.0"
"rollup": "^4.24.0",
"tslib": "^2.8.1"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apparent missing dependency for usage of spread. Not sure exactly why this has arisen now

@@ -16,6 +16,7 @@ COPY packages ./packages
RUN npm ci
RUN npm run build

RUN npm pack --verbose packages/core/
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

core packages is now needed as generated types refer back to @bugsnag/core. Without this the latest publish version of @bugsnag/core will be fetched from npm

@djskinner djskinner requested a review from gingerbenw March 19, 2025 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants