Skip to content

Conversation

@ferdia-sopermaccafraidh-lrn
Copy link
Contributor

@ferdia-sopermaccafraidh-lrn ferdia-sopermaccafraidh-lrn commented Aug 15, 2024

This resolves the issue when running the quickstart guide when installinga as a library from the documentation.

pip install learnosity-sdk[quickstart]

Working now shown below locally in a fresh venv and install from the locally created wheels.

image

Checklist

  • Feature

  • Bug

  • Security

  • Documentation

  • ChangeLog.md updated

  • Tests added

  • All testsuites passed

  • make dist completed successfully

@ferdia-sopermaccafraidh-lrn
Copy link
Contributor Author

@walsh-conor this is my first PR after joining the company, so I could be missing something. If you have a moment to review I would appreciate it! 😄

Please review with caution though due to my new status! 😅

@ferdia-sopermaccafraidh-lrn ferdia-sopermaccafraidh-lrn changed the title Convert docs.quickstart.assessment from namespace to normal package Fix quickstart tutorial when running via pip install learnosity-sdk[quickstart] Aug 21, 2024
Converts assessment from a namespace to a normal module
Copy link
Contributor

@shtrom shtrom left a comment

Choose a reason for hiding this comment

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

LGTM!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants