ballerina-platform / ballerina-central

Ballerina Central Website
Apache License 2.0
106 stars 1 forks source link

Docgen API throws errors when compiling old balas #254

Open anupama-pathirage opened 2 hours ago

anupama-pathirage commented 2 hours ago

@xlight05 cloned issue wso2-enterprise/ballerina-registry#1999 on 2023-03-16:

Description:

$Subject. This occurs with the new change of not generating api docs if it compilation errors with 2201.4.0.

Since ballerina central just uses project API of the latest stable ballerina version to compile the balas uploaded to central to generate api docs, with the above recent change, when an old bala compiled with an older version, is pushed to central, it fails to generate api docs.

We need to think of possible ways to solve this issue.

  1. Pack API docs, triggers and connectors json with the bala and central can consume them. This will remove the need to compile bala from central.

  2. Central needs to compile the package with the pushed ballerina lang dist version. Therefore needs multi-distribution support. This will complicate the implementation a lot.

anupama-pathirage commented 2 hours ago

@xlight05 commented on 2023-03-16:

Reverted the change until we have a proper solution from the central side.

https://github.com/ballerina-platform/ballerina-lang/pull/39901

https://github.com/ballerina-platform/ballerina-lang/pull/39899

https://github.com/ballerina-platform/ballerina-lang/pull/39902