Skip to content

Create integrated documentation#2

Merged
ftomassetti merged 19 commits into
mainfrom
integrated-docs
May 23, 2025
Merged

Create integrated documentation#2
ftomassetti merged 19 commits into
mainfrom
integrated-docs

Conversation

@ftomassetti

@ftomassetti ftomassetti commented May 19, 2025

Copy link
Copy Markdown
Contributor

This is a draft PR, as requested by @enikao .

In this PR we:

  • Reorganize the content currently present at https://lionweb.io, splitting it into multiple pages. We also update the list of contributors, adding Pedro Molina e distinguishing currently active members from members who used to be active
  • Integrate the documentation currently under https://lionweb.io/lionweb-java and https://lionweb.io/lionweb-python
  • Adapt the documentation for LionWeb C#, adding a small tutorial
  • Create some documentation for LionWeb Typescript and the LionWeb Repository

I then plan to add a GitHub action to manage the publication, similarly to what we do here.

The quality of the documentation is not great, but I hope it can serve as a starting point and that we can then incrementally improving it over time.

To check the documentation, run npm run start from the website directory.

Todo

  • Document how we can configure the LionWeb Repository
  • Fix participation list

{
Key = "key-lang",
Version = "0",
Name = "Complaints Language",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not a valid name -- we defined all IKeyed names must be valid programming identifiers

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right

@dslmeinte dslmeinte left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work!

{
public static void Run()
{

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess this is a TODO, right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, I will remove it

Comment thread examples-ts/src/nodes-homogeneous.json Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this a duplicate of the file directly under examples-ts/?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, removing a copy


```shell
# Run the tests
npm run test

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"npm test" does the same thing, and is slightly more ideomatic.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

noted, thank you

@ftomassetti ftomassetti marked this pull request as ready for review May 23, 2025 09:52
@ftomassetti

Copy link
Copy Markdown
Contributor Author

Merging, after discussion during the meeting and because approved by Meinte

@ftomassetti ftomassetti merged commit 9369878 into main May 23, 2025
1 check passed
@enikao enikao deleted the integrated-docs branch July 4, 2025 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants