Since attributes are being encoded now, we should remove the use of setAttributeNode. This was done in https://github.com/canjs/can-util/pull/365 and reverted because some tests were failing in the canjs suite with this change.
We should fix the tests to make this work and revert this revert. I think all that might need to be done is to encode * so that <my-component *viewModel> will work. This was actually done in https://github.com/canjs/can-attribute-encoder/pull/7 also, so that could be re-opened.
Since attributes are being encoded now, we should remove the use of
setAttributeNode. This was done in https://github.com/canjs/can-util/pull/365 and reverted because some tests were failing in the canjs suite with this change.We should fix the tests to make this work and revert this revert. I think all that might need to be done is to encode
*so that<my-component *viewModel>will work. This was actually done in https://github.com/canjs/can-attribute-encoder/pull/7 also, so that could be re-opened.