Skip to content

remove SMLP_manual_v2.pdf and update link in README.md#68

Merged
fbrausse merged 1 commit intomasterfrom
rm-manual-v2
Mar 21, 2026
Merged

remove SMLP_manual_v2.pdf and update link in README.md#68
fbrausse merged 1 commit intomasterfrom
rm-manual-v2

Conversation

@fbrausse
Copy link
Collaborator

As agreed, we'll keep v2 of the manual discussing the upcoming features only in #21 until that's merged.

@fbrausse fbrausse requested a review from zurabksmlp March 20, 2026 11:18
@fbrausse fbrausse mentioned this pull request Mar 20, 2026
21 tasks
@fbrausse fbrausse changed the title remove SMLP_manual_v2.pdf and update link in README.md to point to nlp_text.rebased version of it remove SMLP_manual_v2.pdf and update link in README.md Mar 20, 2026
@fbrausse fbrausse added the metadata-only No changes to source code, test suite or otherwise functionality related items label Mar 20, 2026
Copy link
Collaborator

@zurabksmlp zurabksmlp left a comment

Choose a reason for hiding this comment

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

I understand that only link to SMLP_manual_v2.pdf has changed and no text has changed. I currently do not see a way to verify that the new link works, but I guess the owner will verify it and and I too will verify the link once the change is merged. I approve this change.

@fbrausse
Copy link
Collaborator Author

Hi Zurab, thank you. You can check how the README looks like by clicking the blue link labelled "rm-manual-v2" at the top of this PR. It will bring you to a view of the branch corresponding to this PR, where below the source code tree the README (corresponding to this PR) will be displayed, including the new link.

I'll merge it then.

@fbrausse fbrausse merged commit 5a9be69 into master Mar 21, 2026
@fbrausse fbrausse deleted the rm-manual-v2 branch March 21, 2026 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

metadata-only No changes to source code, test suite or otherwise functionality related items

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants