chore: Release 0.14.0 #182

miquelbeltran · 2024-05-02T20:59:29Z

chore: Release 0.14.0

Description 📝

Purpose: Release 0.14.0
Approach: Merge to master

Note: DO NOT Squash and Merge, but Rebase!

Type of change

Release

Updates

Test plan 🧪

Author to check 👓

Project and all contained modules builds successfully
Self-/dev-tested
Unit/UI/Automation/Integration tests provided where applicable
Code is written to standards
Appropriate documentation written (code comments, internal docs)

Reviewer to check ✔️

Project and all contained modules builds successfully
Change has been dev-/reviewer-tested, where possible
Unit/UI/Automation/Integration tests provided where applicable
Code is written to standards
Appropriate documentation written (code comments, internal docs)

* ci: Create node.js.yml * Include "develop" in the branch list to run CI actions * trigger GitHub actions.

* chore: updated dependencies * chore: update to tslint-react 5.0.0 * fix: mixup between Request and RequestParams * fix: convert Express Request to RequestParams * fix: recreate package-lock to fix express dependency errors * fix: test tap command update * fix: test asserts * refactor: use equal instead of ok

@sumitramanga

Committing @sumitramanga's suggestion Co-authored-by: Sumitra Manga <[email protected]>

Nice! Co-authored-by: Sumitra Manga <[email protected]>

Co-authored-by: Sumitra Manga <[email protected]>

Adding PR, bug report and feature request templates.

* removed tslint * add eslint * add default config * setup default config and npm run command * add command to node.js.yml * set rules to support current codebase * remove unused tslint.json * include sample files in eslint command

* express example: upgrade dependencies and point to subfolder raygun package * using-domains: upgrade dependencies and reference local raygun * using-domains: upgrade dependencies and reference local raygun * add console logs and format using spaces * improved readme * improved readme + replace tabs by spaces * enforce changing API keys to run examples

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [actions/setup-node](https://github.com/actions/setup-node) from 3 to 4. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@v3...v4) --- updated-dependencies: - dependency-name: actions/setup-node dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* removed tslint * add eslint * add default config * setup default config and npm run command * add command to node.js.yml * set rules to support current codebase * remove unused tslint.json * include sample files in eslint command * remove rule no-explicit-any * specify type for timer * remove any from filterKeys * explictly declare that error can be Error, String but still accept any from the end users * specify types in the sync worker * improve the IndexableError type * explain why any is allowed in CustomData * update package lock in sample * Update lib/raygun.sync.worker.ts Co-authored-by: Sumitra Manga <[email protected]> * remove empty line --------- Co-authored-by: Sumitra Manga <[email protected]>

* removed tslint * add eslint * add default config * setup default config and npm run command * add command to node.js.yml * set rules to support current codebase * remove unused tslint.json * include sample files in eslint command * remove rule no-explicit-any * specify type for timer * remove any from filterKeys * explictly declare that error can be Error, String but still accept any from the end users * specify types in the sync worker * improve the IndexableError type * explain why any is allowed in CustomData * remove no-this-alias from eslint config * convert to arrow function to preserve this reference * use arrow functions to remove the need of a this alias * define httpoptions object instead of using this alias * update package lock in sample

* removed tslint * add eslint * add default config * setup default config and npm run command * add command to node.js.yml * set rules to support current codebase * remove unused tslint.json * include sample files in eslint command * remove rule no-explicit-any * specify type for timer * remove any from filterKeys * explictly declare that error can be Error, String but still accept any from the end users * specify types in the sync worker * improve the IndexableError type * explain why any is allowed in CustomData * remove no-this-alias from eslint config * convert to arrow function to preserve this reference * use arrow functions to remove the need of a this alias * define httpoptions object instead of using this alias * setup no-unused-vars lint rule * remove unused var * fixed lib/raygun.batch.ts * fix lib/raygun.offline.ts * cleanup lib/raygun.sync.transport.ts * fix lib/raygun.transport.ts * fix lib/raygun.ts * fix @typescript-eslint/no-unused-vars * fixs in test files * update package lock in sample * cleanup eslint config

* removed tslint * add eslint * add default config * setup default config and npm run command * add command to node.js.yml * set rules to support current codebase * remove unused tslint.json * include sample files in eslint command * remove rule no-explicit-any * specify type for timer * remove any from filterKeys * explictly declare that error can be Error, String but still accept any from the end users * specify types in the sync worker * improve the IndexableError type * explain why any is allowed in CustomData * remove no-this-alias from eslint config * convert to arrow function to preserve this reference * use arrow functions to remove the need of a this alias * define httpoptions object instead of using this alias * add examples to prettier command * add prettier job to node js ci job * script fix * setup no-unused-vars lint rule * remove unused var * fixed lib/raygun.batch.ts * fix lib/raygun.offline.ts * cleanup lib/raygun.sync.transport.ts * fix lib/raygun.transport.ts * fix lib/raygun.ts * fix @typescript-eslint/no-unused-vars * fixs in test files * update package lock in sample * cleanup eslint config * config files * run npm run prettier * Update .github/workflows/node.js.yml Co-authored-by: Sumitra Manga <[email protected]> * fix workflow * add comments to the package.json * add missing files * remove fetch-depth: 0 --------- Co-authored-by: Sumitra Manga <[email protected]>

* removed tslint * add eslint * add default config * setup default config and npm run command * add command to node.js.yml * set rules to support current codebase * remove unused tslint.json * include sample files in eslint command * remove rule no-explicit-any * specify type for timer * remove any from filterKeys * explictly declare that error can be Error, String but still accept any from the end users * specify types in the sync worker * improve the IndexableError type * explain why any is allowed in CustomData * remove no-this-alias from eslint config * convert to arrow function to preserve this reference * use arrow functions to remove the need of a this alias * define httpoptions object instead of using this alias * add examples to prettier command * add prettier job to node js ci job * script fix * setup no-unused-vars lint rule * remove unused var * fixed lib/raygun.batch.ts * fix lib/raygun.offline.ts * cleanup lib/raygun.sync.transport.ts * fix lib/raygun.transport.ts * fix lib/raygun.ts * fix @typescript-eslint/no-unused-vars * fixs in test files * update package lock in sample * cleanup eslint config * config files * run npm run prettier * chore: document why no-var-requires ignore is necessary update comment

* removed tslint * add eslint * add default config * setup default config and npm run command * add command to node.js.yml * set rules to support current codebase * remove unused tslint.json * include sample files in eslint command * remove rule no-explicit-any * specify type for timer * remove any from filterKeys * explictly declare that error can be Error, String but still accept any from the end users * specify types in the sync worker * improve the IndexableError type * explain why any is allowed in CustomData * remove no-this-alias from eslint config * convert to arrow function to preserve this reference * use arrow functions to remove the need of a this alias * define httpoptions object instead of using this alias * add examples to prettier command * add prettier job to node js ci job * script fix * setup no-unused-vars lint rule * remove unused var * fixed lib/raygun.batch.ts * fix lib/raygun.offline.ts * cleanup lib/raygun.sync.transport.ts * fix lib/raygun.transport.ts * fix lib/raygun.ts * fix @typescript-eslint/no-unused-vars * fixs in test files * update package lock in sample * cleanup eslint config * config files * run npm run prettier * chore: document why no-var-requires ignore is necessary update comment * fix: #158 configure no-undef eslint rule

* refactor: #158 configure no-unreachable eslint rule * refactor: #158 remove no-useless-catch eslint rule

* refactor: #158 configure no-unreachable eslint rule * refactor: #158 remove no-useless-catch eslint rule * set version 0.14.0 * update changelog * Update package.json Co-authored-by: Sumitra Manga <[email protected]> * add Error reporting to keywords * update license copyright year * updated URL * added more keywords * moar keywords --------- Co-authored-by: Sumitra Manga <[email protected]>

sumitramanga

I've tested and code reviewed during the previous release branch (#179) which piggy backed off of all the changes in develop branch that needs to be merged in. So I'll go ahead and approve this PR

Great work team!! 💪🏽

miquelbeltran · 2024-05-02T21:32:06Z

ugh, first time I see this :)

sumitramanga · 2024-05-02T21:38:35Z

@miquelbeltran that happens when a commit is pushed to the branch 😅 But that won't happen when merging master into your branch has no conflicts

sumitramanga

Testing went well! LGTM 🚀

TheRealAgentK

LGTM

PanosNB

Great work team! Really enjoyed the great Software Engineering process structure going in there! Just a few points to consider and a pedantic task

examples/express-sample/app.js

lib/raygun.messageBuilder.ts

lib/raygun.offline.ts

package.json

PanosNB

Thanks @miguel Beltran! It seems that the recursive object error is unlikely to surface easily. So yes, you can proceed with this release but let's ensure we protect against recursive objects in the next release please. Same for the other one. And thanks for taking care of renaming MindscapeHQ to Raygun!

TheRealAgentK

Re-approval after additional commit.

sumitramanga

LGTM! Example apps are working well

miquelbeltran and others added 24 commits April 24, 2024 08:23

ci: Create node.js.yml (#142)

783b3b2

* ci: Create node.js.yml * Include "develop" in the branch list to run CI actions * trigger GitHub actions.

Merge branch 'master' into develop

5ea3a32

Adding PR, bug report and feature request templates.

60eccc4

Update pull_request_template.md

8f06ec3

Committing @sumitramanga's suggestion Co-authored-by: Sumitra Manga <[email protected]>

Update .github/bug_report.md

cfe701a

Nice! Co-authored-by: Sumitra Manga <[email protected]>

Update .github/bug_report.md

8b458bd

Co-authored-by: Sumitra Manga <[email protected]>

Merge pull request #154 from MindscapeHQ/feature/#147-GH-Templates

aec342f

Adding PR, bug report and feature request templates.

ci: #145 eslint setup (#156)

1650c25

* removed tslint * add eslint * add default config * setup default config and npm run command * add command to node.js.yml * set rules to support current codebase * remove unused tslint.json * include sample files in eslint command

ci: add PR title checks (#167)

ebdc289

ci: Update dependabot.yml (#168)

3bf2cc0

chore: Move pr.yml to workflows (#175)

23c899b

refactor: #158 configure no-unreachable eslint rule (#177)

5df7c14

refactor: #158 remove no-useless-catch eslint rule (#178)

881365a

* refactor: #158 configure no-unreachable eslint rule * refactor: #158 remove no-useless-catch eslint rule

miquelbeltran changed the title ~~release: 0.14.0~~ chore: 0.14.0 May 2, 2024

miquelbeltran requested review from a team, TheRealAgentK, PanosNB and sumitramanga and removed request for a team May 2, 2024 21:00

miquelbeltran marked this pull request as ready for review May 2, 2024 21:01

miquelbeltran changed the title ~~chore: 0.14.0~~ chore: Release 0.14.0 May 2, 2024

sumitramanga previously approved these changes May 2, 2024

View reviewed changes

chore: Update package-lock.json (#180)

e86a771

miquelbeltran dismissed sumitramanga’s stale review via e86a771 May 2, 2024 21:30

chore: update package-lock.json (#183)

d7d0593

miquelbeltran requested a review from sumitramanga May 2, 2024 21:31

sumitramanga previously approved these changes May 2, 2024

View reviewed changes

TheRealAgentK previously approved these changes May 2, 2024

View reviewed changes

PanosNB reviewed May 2, 2024

View reviewed changes

examples/express-sample/app.js Show resolved Hide resolved

lib/raygun.messageBuilder.ts Show resolved Hide resolved

lib/raygun.offline.ts Show resolved Hide resolved

package.json Outdated Show resolved Hide resolved

miquelbeltran mentioned this pull request May 3, 2024

chore: change author name #185

Merged

11 tasks

chore: change author name (#185)

da8b899

miquelbeltran dismissed stale reviews from TheRealAgentK and sumitramanga via da8b899 May 3, 2024 06:10

TheRealAgentK mentioned this pull request May 3, 2024

Look into potential cyclic objects #186

Closed

PanosNB approved these changes May 3, 2024

View reviewed changes

TheRealAgentK approved these changes May 3, 2024

View reviewed changes

miquelbeltran mentioned this pull request May 4, 2024

feat: #146 async send support #181

Merged

12 tasks

sumitramanga approved these changes May 6, 2024

View reviewed changes

miquelbeltran merged commit d3b2e9c into master May 6, 2024
14 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: Release 0.14.0 #182

chore: Release 0.14.0 #182

miquelbeltran commented May 2, 2024 •

edited by sumitramanga

Loading

sumitramanga left a comment

miquelbeltran commented May 2, 2024

sumitramanga commented May 2, 2024

sumitramanga left a comment

TheRealAgentK left a comment

PanosNB left a comment

PanosNB left a comment

TheRealAgentK left a comment

sumitramanga left a comment

chore: Release 0.14.0 #182

chore: Release 0.14.0 #182

Conversation

miquelbeltran commented May 2, 2024 • edited by sumitramanga Loading