Skip to content

Refactoring of tools.md#330

Merged
frankier merged 3 commits intocoderefinery:mainfrom
mmesiti:tools-refactor-spring2026
Mar 20, 2026
Merged

Refactoring of tools.md#330
frankier merged 3 commits intocoderefinery:mainfrom
mmesiti:tools-refactor-spring2026

Conversation

@mmesiti
Copy link
Collaborator

@mmesiti mmesiti commented Mar 18, 2026

Addresses #310.

Some repetitions between tabs, but still shorter.
Mention "general features" of HTML static site generators.

Note: Still missing: table with comparison between different documentation types, as suggested by @ashwinvis in #310 (comment).

This will be added in a separate PR, we might need to think a little more about it.

@mmesiti mmesiti self-assigned this Mar 18, 2026
@mmesiti mmesiti requested a review from ashwinvis March 18, 2026 19:10
@ashwinvis
Copy link
Contributor

Looks pretty good already.

@frankier
Copy link
Collaborator

frankier commented Mar 19, 2026

LGTM, maybe just merge it and make a follow up issue?

Some repetitions between tabs, but still shorter.
Mention "general features" of HTML static site generators.
@mmesiti mmesiti force-pushed the tools-refactor-spring2026 branch from 3329ed5 to 56815b8 Compare March 19, 2026 15:02
@mmesiti
Copy link
Collaborator Author

mmesiti commented Mar 19, 2026

LGTM, maybe just merge it and make a follow up issue?

This is very wise, if I don't manage to finish by today I'll do it

@mmesiti mmesiti marked this pull request as ready for review March 19, 2026 16:19
@mmesiti
Copy link
Collaborator Author

mmesiti commented Mar 19, 2026

Update: decided to defer the implementation of the comparison table (will be another PR), but I am not sure myself about the results, so I think it makes sense to wait and do this once we all agree on the result (will open next draft PR shortly)

@mmesiti mmesiti mentioned this pull request Mar 19, 2026
@frankier frankier merged commit f1f8492 into coderefinery:main Mar 20, 2026
2 checks passed
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