I'm not sure if I'm okay with dropping the error message - I think that decorators that won't properly work in ES3 should get an error.

Daniel Rosenwasser (@DanielRosenwasser) Are you talking about a specific error? Part of this change is to have the emit for method decorators in ES3 not include the property descriptor, just as with decorators on property declarations.

but this will have a different behavior for ES3/ES5 for method decorators with no errors.

Yes, if method decorators won't work properly, then you should warn users that method decorators are only supported in ES5 or higher.

Daniel Rosenwasser (@DanielRosenwasser), In this change method decorators will work in the same way that decorators for property declarations work. No descriptor will be provided nor accepted. There will be no error.

Daniel Rosenwasser (@DanielRosenwasser), Mohamed Hegazy (@mhegazy) Any further concerns/discussion with this PR?

👍
I am still not sure about the error though

The error that I removed no longer applies. The only difference between decorators in ES3 and ES5 is that ES3 decorators cannot take and don't observe a descriptor.

Ron Buckton (@rbuckton) I'll need to ask a few questions about this offline.