Skip to content

Deprecate the Resource class #24499

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.

Sign up for GitHub

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

Jump to bottom
Closed
sethladd opened this issue Oct 5, 2015 · 5 comments
Closed

Deprecate the Resource class #24499

sethladd opened this issue Oct 5, 2015 · 5 comments
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries.
Milestone
1.13

Comments

@sethladd
Copy link
Contributor

sethladd commented Oct 5, 2015

We believe that a package will be published that can contain the Resource class. Therefore, we can deprecate the Resource class in the SDK.

Requires the new package to be published, so we can link to it from the deprecation notice.
@sethladd sethladd added the area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. label Oct 5, 2015
@sethladd sethladd added this to the 1.13 milestone Oct 5, 2015
@sethladd
Copy link
Contributor Author

sethladd commented Oct 5, 2015

https://chromiumcodereview.appspot.com/1385933002/ is the template, still needs the link.

@sethladd sethladd mentioned this issue Oct 5, 2015
Closed
@mit-mit
Copy link
Member

mit-mit commented Oct 6, 2015

Per discussion with @floitschG, we believe we should remove the resource class in 1.13. It was marked experimental in 1.12, and the actual work for a customer to migrate to the new package is very, very minimal.

@mit-mit mit-mit closed this as completed Oct 6, 2015
@sethladd
Copy link
Contributor Author

sethladd commented Oct 6, 2015

I have deprecated the class, marked for removal in 1.14. See https://codereview.chromium.org/1385933002

@kasperpeulen
Copy link

I don't understand how I should use the Resource class now. The dart package is deprecated and the pub package is pending.

@mit-mit
Copy link
Member

mit-mit commented Dec 28, 2015

We should have some more details on this (incl. new APIs) in the next few weeks. Stay tuned!

CC @kevmoo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries.
Projects
None yet
Milestone
1.13
Development

No branches or pull requests
3 participants
@sethladd @kasperpeulen @mit-mit