Skip to content

doc: Extend README#10

Merged
erikap merged 23 commits intomu-semtech:masterfrom
mirdono:chore/extend-readme
Feb 17, 2026
Merged

doc: Extend README#10
erikap merged 23 commits intomu-semtech:masterfrom
mirdono:chore/extend-readme

Conversation

@mirdono
Copy link
Contributor

@mirdono mirdono commented Feb 4, 2026

This PR extends the service's draft README with concrete contents following the structure defined by the The Documentation System. This PR builds upon the improvements initially proposed in #6 and thereby supersedes that PR.

In addition this PR also fixes several typos that were encountered throughout the codebase while writing the documentation.

TODO

  • Document defining and using scopes

Related ticket(s)

  • LBRON-334

@Rahien
Copy link

Rahien commented Feb 6, 2026

I think the work you did here is nothing short of amazing! Is it worth pointing out that you can set the env variable LISP_DYNAMIC_SPACE_SIZE? In an older version of sparql-parser, this wasn't set and it had trouble dealing with a setup with many quite large queries.

@mirdono
Copy link
Contributor Author

mirdono commented Feb 6, 2026

I think the work you did here is nothing short of amazing! Is it worth pointing out that you can set the env variable LISP_DYNAMIC_SPACE_SIZE? In an older version of sparql-parser, this wasn't set and it had trouble dealing with a setup with many quite large queries.

Thanks I overlooked there is actually one environment variable for this service. I added a section in Reference.

@erikap
Copy link
Member

erikap commented Feb 17, 2026

Thanks a lot @mirdono ! 🙏

@erikap erikap merged commit f2bcfe9 into mu-semtech:master Feb 17, 2026
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.

4 participants