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

document provenance requirements for new/existing code #968

Open
petrelharp opened this issue Apr 22, 2020 · 3 comments
Open

document provenance requirements for new/existing code #968

petrelharp opened this issue Apr 22, 2020 · 3 comments
Assignees
Milestone

Comments

@petrelharp
Copy link
Contributor

With the new provenance code there are some additional requirements for adding new methods or modifying existing ones, as I'm discovering over in #946. In figuring this out it'd be very helpful to have a short explainer and/or checklist of what needs to happen to interface with the provenance machinery. For instance: an .asdict() method... ? Right now I'm going on "what do errors in running tests tell me", and I'm starting to get a sense for how things work (which seems very nice!), but the big picture would really help. Apologies if this exists somewhere and I have missed it.

@benjeffery
Copy link
Member

Agreed! I should have done this as part of the initial work. I'll self-assign this.

@benjeffery benjeffery self-assigned this Apr 22, 2020
@petrelharp
Copy link
Contributor Author

Great! Thanks.

@benjeffery benjeffery added this to the Release 1.0 milestone Aug 24, 2020
@jeromekelleher
Copy link
Member

I'm going to drop this out of the 1.0 milestone because it's more of an internal development thing and not really user facing.

@jeromekelleher jeromekelleher modified the milestones: Release 1.0, 1.1 Mar 8, 2021
@jeromekelleher jeromekelleher modified the milestones: 1.2.0, 1.3.0 Feb 23, 2022
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

No branches or pull requests

3 participants