Skip to content

feat(error.js): Add cwd error property#565

Merged
sindresorhus merged 3 commits intosindresorhus:mainfrom
homersimpsons:feature/error-cwd
Jul 26, 2023
Merged

feat(error.js): Add cwd error property#565
sindresorhus merged 3 commits intosindresorhus:mainfrom
homersimpsons:feature/error-cwd

Conversation

@homersimpsons
Copy link
Contributor

@homersimpsons homersimpsons commented Jul 14, 2023

Fixes #564

(I couldn't find any contribution guidelines, so I hope everything is okay)

@ehmicky I let you review this

@ehmicky
Copy link
Collaborator

ehmicky commented Jul 14, 2023

Thanks for the contribution @homersimpsons!

We should also add:

  • some TypeScript types in index.d.ts
  • some type tests in index.test-d.ts
  • some documentation in the README.md
  • copying that documentation in index.d.ts

Thanks again!

@homersimpsons
Copy link
Contributor Author

Thank you for the feedback, I will certainly do this on Monday 24th

@ehmicky
Copy link
Collaborator

ehmicky commented Jul 14, 2023

Thanks, please take your time!

Copy link
Collaborator

@ehmicky ehmicky left a comment

Choose a reason for hiding this comment

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

This looks good to me! Thanks @homersimpsons.
What do you think @sindresorhus?

@sindresorhus sindresorhus merged commit f57fdec into sindresorhus:main Jul 26, 2023
@homersimpsons homersimpsons deleted the feature/error-cwd branch July 27, 2023 16:35
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.

Add cwd to error

3 participants