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

chore: replace several instances of any #2264

Merged
merged 3 commits into from Nov 8, 2019

Conversation

SebastiaanNijland
Copy link
Contributor

  • Rpeplace any in several places with the actual types
  • Improve checks of event / exception to prevent TypeErrors (this was needed to be able to rewrite (event as any) to just event)

Before submitting a pull request, please take a look at our
Contributing guidelines and verify:

  • If you've added code that should be tested, please add tests.
  • Ensure your code lints and the test suite passes (yarn lint) & (yarn test).

@kamilogorek kamilogorek merged commit f5e1e51 into getsentry:master Nov 8, 2019
@kamilogorek
Copy link
Contributor

Thanks!

HazAT added a commit that referenced this pull request Nov 15, 2019
* master:
  release: 5.9.0
  feat: Introduce mode option for global promise handler in node (#2312)
  release: 5.8.0
  misc: 5.8.0 changelog update
  fix(utils/object): fix normalizing error from null (#2305)
  doc: Link to discord, not IRC (#2304)
  fix: Skip empty filename in frames and fallback to location
  chore: Linter auto-fixes
  misc: 5.8.0 changelog
  chore: replace several instances of any (#2264)
  feat: 429 http code handling in node/browser transports
  ref: Remove dom references from utils for old TS and env interop
  fix: Gracefuly handle incorrect input from onerror
  feat: Make sure that Debug integration is always setup as the last one

# Conflicts:
#	lerna.json
#	packages/browser/package.json
#	packages/browser/src/version.ts
#	packages/core/package.json
#	packages/hub/package.json
#	packages/integrations/package.json
#	packages/minimal/package.json
#	packages/node/package.json
#	packages/node/src/version.ts
#	packages/utils/package.json
#	packages/utils/src/misc.ts
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.

None yet

2 participants