microsoft / typespec

https://typespec.io/
MIT License
4.54k stars 220 forks source link

http-client-java, do not prefix com. in namespace #5084

Closed weidongxu-microsoft closed 1 week ago

weidongxu-microsoft commented 1 week ago

Also remove the Cadl. in local test. Replaced it with TspTest (we don't want to call it TypeSpec. as it could conflict with TypeSpec libs).

It would not affect Azure SDK which always has namespace in emitter option. It only affect SDK that does not have this option (basically, test case, and unbranded).

It prepares for the Namespace refactor, as we should not modify the TypeSpec Namespace. https://github.com/Azure/autorest.java/issues/2968

azure-sdk commented 1 week ago

No changes needing a change description found.

azure-sdk commented 1 week ago

You can try these changes here

🛝 Playground 🌐 Website 📚 Next docs
haolingdong-msft commented 1 week ago

Will this cause breaking changes to released lib?

weidongxu-microsoft commented 1 week ago

Will this cause breaking changes to released lib?

No. Azure lib should all have the "namespace" option in tspconfig.

I will double check with a test sync.