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

WIP: Tagging for code level of user exposure #371

Closed
wants to merge 3 commits into from

Conversation

rileyjmurray
Copy link
Collaborator

This work-in-progress PR means to address #354.

The plan is to keep this PR open and for each of us to periodically update this branch. @enielse has gotten us started with tagging a few modules. Although the process has just begun, we've already found a module that has minimal user exposure and that's "forsaken"! (Ripe for the removing!)

Tips for working on this:

  • Pull from GitHub before making changes. There's a chance someone else might have already pushed some work you don't see on your local repo.
  • If you're about to spend a while on this (say, more than 15 minutes) then it would probably be a good idea to post here and explain what (sub) folders you're looking at. That way we don't duplicate effort.

CC: @sserita and @coreyostrove.

enielse and others added 3 commits November 14, 2023 13:02
Add tagging to the top-levels of the rest of algorithms module.
Temporarily disable the on-push unit testing in anticipation for many comment-only commits getting pushed to this branch.
@coreyostrove coreyostrove linked an issue Nov 29, 2023 that may be closed by this pull request
@sserita
Copy link
Contributor

sserita commented Dec 19, 2023

This is obsolete in favor of #385, from the ionq fork.

@sserita sserita closed this Dec 19, 2023
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.

Developer infrastructure: mark code with level of user-exposure
3 participants