Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

add basic pico nodes #329

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

add basic pico nodes #329

wants to merge 5 commits into from

Conversation

dstrande
Copy link
Contributor

@dstrande dstrande commented Oct 13, 2023 •
edited
Loading

Description

  • I've included a concise description of what each node does

Styleguide

  • My node adheres to the styleguide for Flojoy nodes

Docs

  • I've submitted a PR for a documentation page for the new node(s) that contains usage examples (see docs.flojoy.io)

Testing

  • This PR includes a unit test (example here and/or ideally a screenshot of the node's output on an example app.

@linear
Copy link

linear bot commented Oct 13, 2023

NOD-62 Picoscope Nodes

@flojoy-bot flojoy-bot bot mentioned this pull request Oct 13, 2023
Draft
@dstrande dstrande requested review from itsjoeoui and 39bytes October 16, 2023 16:22
@dstrande dstrande enabled auto-merge October 24, 2023 14:31
@dstrande dstrande requested a review from 39bytes October 24, 2023 14:31
@dstrande dstrande mentioned this pull request Oct 24, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@itsjoeoui itsjoeoui Awaiting requested review from itsjoeoui

@39bytes 39bytes Awaiting requested review from 39bytes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dstrande @39bytes