-
Notifications
You must be signed in to change notification settings - Fork 105
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
Replace all new Error(...)
with new DwnError(...)
#387
Comments
picking this up 🙂 |
@tux7P Yay! I'll assign it to you now. I'd also like to note that Hacktoberfest officially kicks off on October 1st. Contributions made before the event may not be eligible for the challenge. However, feel free to start working on it, but I'd recommend holding off on submitting your PR until the event begins.
|
Hi @EbonyLouis @thehenrytsai |
@tux7P Hey do you still want to work on this issue? |
@FroostyCodes it's all yours, assigning it to you now, thanks! 🥳 |
@EbonyLouis can i work on this if the current assignee is inactive ? Thanks |
@Tushar98644 are you still wanting to work on this? if not, I can pick it up. |
@flothjl thanks, you can work on this |
Background:
currently we have
new DwnError(...)
andnew Error(...)
throughout the codeObjective:
new Error(...)
withnew DwnError(...)
src/core/dwn-error.ts
Picking Up This Issue:
Questions:
#hack-together
channel and collaborate with a buddy.#getting-started
channel.Resources:
Remember, communication is key! If you have any questions or face any challenges, we're here to help so please don't hesitate to reach out.
Good Luck! 🍁
The text was updated successfully, but these errors were encountered: