-
Notifications
You must be signed in to change notification settings - Fork 535
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
Add breaking.md #9701
Add breaking.md #9701
Conversation
|
||
### Doing operations not allowed on deleted sub directory | ||
Users will not be allowed to do operations on a deleted directory. Users can subscribe to `disposed` event to know if a sub directory is deleted. Accessing deleted sub directory will throw `UsageError` exception now. | ||
|
||
### IDirectory extends IDisposable | ||
IDirectory has started extending IDisposable. This means that users implementing the IDirectory interface needs to implement IDisposable too now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just to confirm, is the plan to change the implements Partial<IDisposable>
to implements IDisposable
in the future?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes.
@@ -18,10 +18,14 @@ There are a few steps you can take to write a good change note and avoid needing | |||
|
|||
## 0.58 Upcoming changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@heliocliu I would recommend changing this to not say 0.58, maybe "Upcoming changes (ETA 0.59)" or something. Or just "Upcoming changes" and as part of the release clear the section out every time...?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I definitely wanted to avoid the step of needing to clear them out, but it's true that having a version associated with the upcoming change has been confusing for people :/ But yeah the ease of understanding probably outweighs needing to delete stuff occasionally there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why would you need to clear them out?
Add in upcoming changes about IDirectory extending IDisposable.