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

feat(NODE-3419): define MongoRuntimeError children #2893

Merged
merged 3 commits into from
Jul 13, 2021

Conversation

andymina
Copy link
Contributor

@andymina andymina commented Jul 8, 2021

Description

What changed?
Classes added:

  • MongoURIError
  • MongoCompressionError
  • MongoDecompressionError
  • MongoTransactionError
  • MongoExpiredSessionError
  • MongoKerberosError
  • MongoStreamError
  • MongoChangeStreamError
  • MongoGridFSStreamError
  • MongoGridFSChunkError
  • MongoResourceClosedError
  • MongoServerClosedError
  • MongoStreamClosedError
  • MongoTopologyClosedError

@andymina andymina added the wip label Jul 8, 2021
@andymina andymina self-assigned this Jul 8, 2021
@andymina andymina requested a review from W-A-James July 8, 2021 19:21
@andymina andymina removed the wip label Jul 8, 2021
@andymina andymina marked this pull request as ready for review July 8, 2021 19:21
Copy link
Contributor

@W-A-James W-A-James left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏿

@W-A-James W-A-James added the Team Review Needs review from team label Jul 8, 2021
@andymina andymina changed the title feat(NODE-3419): defined MongoRuntimeError children feat(NODE-3419): defined MongoRuntimeError children pt 2 and 3 Jul 8, 2021
Copy link
Contributor

@emadum emadum left a comment

Choose a reason for hiding this comment

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

I think this documentation would read better if we replaced the boilerplate An error generated when an attempt is made to ... to something more succinct like Thrown when attempting to ..., thoughts? Nit, but could we also make the line length used for wrapping consistent across all the docs?

src/error.ts Outdated
}

/**
* An error generated when teh driver fails to decompress
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* An error generated when teh driver fails to decompress
* An error generated when the driver fails to decompress

src/error.ts Outdated
}

/**
* An error generated when an attempt to access a resource
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* An error generated when an attempt to access a resource
* An error generated when an attempt is made to access a resource

@nbbeeken nbbeeken changed the base branch from NODE-3399/Implement-New-Error-hierarchy to 4.0 July 9, 2021 14:20
Copy link
Contributor

@nbbeeken nbbeeken left a comment

Choose a reason for hiding this comment

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

Seconded on Eric's comments, otherwise LGTM (also rebase needed)

@andymina andymina requested a review from emadum July 9, 2021 21:20
Copy link
Contributor

@emadum emadum left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Contributor

@dariakp dariakp left a comment

Choose a reason for hiding this comment

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

LGTM

@andymina andymina force-pushed the NODE-3419/define-MongoRuntimeError-children branch from f941c79 to 2ea2e84 Compare July 12, 2021 15:48
@andymina andymina changed the title feat(NODE-3419): defined MongoRuntimeError children pt 2 and 3 feat(NODE-3419): define MongoRuntimeError children Jul 13, 2021
@W-A-James W-A-James merged commit eadeb01 into 4.0 Jul 13, 2021
@W-A-James W-A-James deleted the NODE-3419/define-MongoRuntimeError-children branch July 13, 2021 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team Review Needs review from team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants