Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use new {tinkr} method $get_protected() #155

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

zkamvar
Copy link
Contributor

@zkamvar zkamvar commented May 27, 2024

This PR will do three things:

  1. first test to make sure that the dev version of {tinkr} continues to work
    with the current version of {pegboard}
  2. implement the $get_protected() method to prepare for changes coming in
    {tinkr} 1.0.0.
  3. only test get_lesson() on CI to avoid weird SSL issues in tests (which otherwise work fine interactively).

Only testing on CI because there are weird SSL issues popping up that
makes it slower than molases
I don't _believe_ than anything has failed, but i want to make sure.

This will allow us to move forward with the `$get_protected()` method
so that {tinkr} can implement more control over protected and
unprotected nodes without needing to constantly split nodes.
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.

1 participant