Skip to content
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.

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

[Improvement]: Add README.md as the primary documentation file for library packages #42981

Closed
azinneera opened this issue Jun 25, 2024 · 2 comments · Fixed by #42689, #43201, #43497 or #43618 · May be fixed by #43642
Closed

[Improvement]: Add README.md as the primary documentation file for library packages #42981

azinneera opened this issue Jun 25, 2024 · 2 comments · Fixed by #42689, #43201, #43497 or #43618 · May be fixed by #43642

Comments

@azinneera
Copy link
Contributor

Description

Currently, the documentation is captured in two files: Package.md for package documentation and additionally, a Module.md for module-specific documentation. This has to be improved to reduce to one file, and since most of the other platforms like GitHub supports README.md by default for documentation, we should use the same name.

Moreover, the user must have the option to specify a custom .md file for documentation. A new field under the [package] table in the Ballerina.toml file must be introduced for this.

Describe your problem(s)

No response

Describe your solution(s)

No response

Related area

-> Compilation

Related issue(s) (optional)

No response

Suggested label(s) (optional)

No response

Suggested assignee(s) (optional)

No response

@azinneera azinneera added this to the 2201.10.0 milestone Jun 25, 2024
@ballerina-bot ballerina-bot added needTriage The issue has to be inspected and labeled manually userCategory/Compilation labels Jun 25, 2024
@azinneera azinneera removed needTriage The issue has to be inspected and labeled manually userCategory/Compilation labels Jul 2, 2024
@azinneera azinneera self-assigned this Jul 2, 2024
@azinneera azinneera moved this from BackLog to In Progress in Ballerina Team Main Board Jul 2, 2024
@gayaldassanayake gayaldassanayake moved this from In Progress to On Hold in Ballerina Team Main Board Jul 9, 2024
@azinneera azinneera moved this from On Hold to In Progress in Ballerina Team Main Board Jul 23, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Ballerina Team Main Board Aug 1, 2024
@azinneera azinneera removed this from the 2201.10.0 milestone Aug 12, 2024
@azinneera azinneera reopened this Aug 14, 2024
@azinneera azinneera moved this from Done to In Progress in Ballerina Team Main Board Aug 14, 2024
@azinneera azinneera moved this from In Progress to On Hold in Ballerina Team Main Board Oct 4, 2024
@azinneera
Copy link
Contributor Author

azinneera commented Oct 7, 2024

The design that was finalized on 10/09/2024 is as follows:

[package]  # Describes the package and default module
org = "samjs"
name = "mylib"
version = "0.1.0"
description = "" # Package and default module description 
distribution = "2201.11.0"
readme = "" # Package and default module document

[[package.modules]]
name = "mylib.foo" # valid only for non-default modules
description = ""
export = true # by default false
readme = ""

[[package.modules]]
name = "mylib" # default module is disallowed
description = ""

The implementation was kept on hold due to the prioritization of dev rel work.

@azinneera azinneera moved this from On Hold to In Progress in Ballerina Team Main Board Oct 7, 2024
@azinneera azinneera moved this from In Progress to On Hold in Ballerina Team Main Board Oct 14, 2024
@azinneera azinneera moved this from In Progress to On Hold in Ballerina Team Main Board Oct 18, 2024
@azinneera
Copy link
Contributor Author

With this change, we need to change the structure of the package.json in the BALA file to capture module information as well. This is currently being designed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment