Skip to content

Commit f9c978e

Browse files
Nikola-3ngrujicmattcompiles
authored
Add sentry support (#9598)
* write version dynamically to rust exposed parcel to sentry via env var * added tag support via env vars * remove redundant panic call * added conditional compilation, only compiles sentry on canary builds, updated pipelines and improved logging of tags * remove panic used for testing * respond to PR comments * fix flow error, add empty stub for non-canary builds * missing change from upstream * revert dev-release with sentry * add napi import to stub * reintroduce arc for type compatibility * fix flow error --------- Co-authored-by: ngrujic <[email protected]> Co-authored-by: Matt Jones <[email protected]>
1 parent 91b47ff commit f9c978e

File tree

15 files changed

+1120
-64
lines changed

15 files changed

+1120
-64
lines changed

.github/workflows/canary-release.yml

+1
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ jobs:
1515
uses: ./.github/workflows/release.yml
1616
secrets: inherit
1717
with:
18+
profile: canary
1819
release-command: |
1920
yarn canary:release --summary-file
2021
node scripts/tag-release.mjs --tag nightly

.github/workflows/dev-release.yml

+1
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,6 @@ jobs:
1212
uses: ./.github/workflows/release.yml
1313
secrets: inherit
1414
with:
15+
profile: canary
1516
release-command: yarn dev:release
1617
type: dev

.github/workflows/release.yml

+5-5
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ jobs:
5353
if: ${{ matrix.target == 'aarch64-apple-darwin' }}
5454
run: sudo rm -Rf /Library/Developer/CommandLineTools/SDKs/*;
5555
- name: Build native packages
56-
run: yarn build-native-release
56+
run: yarn build-native-${{ input.profile || 'release' }}
5757
env:
5858
RUST_TARGET: ${{ matrix.target }}
5959
- name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034
@@ -88,7 +88,7 @@ jobs:
8888
override: true
8989
- uses: bahmutov/[email protected]
9090
- name: Build native packages
91-
run: yarn build-native-release
91+
run: yarn build-native-${{ input.profile || 'release' }}
9292
- name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034
9393
run: strip packages/*/*/*.node
9494
- name: Upload artifacts
@@ -133,7 +133,7 @@ jobs:
133133
with:
134134
shared-key: ${{ matrix.target }}
135135
- name: Build native packages
136-
run: yarn build-native-release
136+
run: yarn build-native-${{ input.profile || 'release' }}
137137
env:
138138
RUST_TARGET: ${{ matrix.target }}
139139
CFLAGS: ${{ matrix.cflags }}
@@ -186,7 +186,7 @@ jobs:
186186
target: ${{ matrix.target }}
187187
- uses: bahmutov/[email protected]
188188
- name: Build native packages
189-
run: yarn build-native-release
189+
run: yarn build-native-${{ input.profile || 'release' }}
190190
env:
191191
RUST_TARGET: ${{ matrix.target }}
192192
CFLAGS: ${{ matrix.cflags }}
@@ -217,7 +217,7 @@ jobs:
217217
fetch-depth: 0
218218
- uses: bahmutov/[email protected]
219219
- name: Build native packages
220-
run: yarn build-native-release
220+
run: yarn build-native-${{ input.profile || 'release' }}
221221
- name: Download artifacts
222222
uses: actions/download-artifact@v3
223223
with:

0 commit comments

Comments
 (0)