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

Annotate method inputs and outputs using typing #39

Closed
cmungall opened this issue Jul 2, 2022 · 1 comment
Closed

Annotate method inputs and outputs using typing #39

cmungall opened this issue Jul 2, 2022 · 1 comment
Milestone

Comments

@cmungall
Copy link
Collaborator

cmungall commented Jul 2, 2022

For example:

def parse(input):
    """
    Parse a set of KGCL command separated by next-line operator.

    Returns instantiated dataclass objects from model.kgcl.
    """
    statements = input.splitlines()
    parsed = []

    for s in statements:
        parsed.append(parse_statement(s))
    return parsed

The arguments and the output should be typed

@cmungall cmungall added this to the First release milestone Jul 2, 2022
cmungall added a commit to INCATools/kgcl that referenced this issue Jul 2, 2022
@cmungall cmungall changed the title Use modern python idioms such as typing Annotate method inputs and outputs using typing Jul 5, 2022
cmungall added a commit to INCATools/kgcl that referenced this issue Jul 5, 2022
cmungall added a commit to INCATools/kgcl that referenced this issue Jul 5, 2022
* Adding CLI to pyproject, see INCATools/kgcl-rdflib#24

* Added some typing, see INCATools/kgcl-rdflib#39

* Test refactoring.

Using subfolders to differentiate different kinds of tests.
Adding a test for grammar INCATools/kgcl-rdflib#41
Adding tests for string issue INCATools/kgcl-rdflib#36

* Using correct paths for imports, see INCATools/kgcl-rdflib#40

* Adding tests for grammar and rendering. Fixes INCATools/kgcl-rdflib#41

* Adding click tests.
Making CLI more clig-compliant - changing some arguments to options
Fixes INCATools/kgcl-rdflib#45

* Deprecating duplicative slots that store the type of the entity.
See INCATools/kgcl-rdflib#44

* Adding collection of test cases that can be reused for testing different subcomponents:
parsing, diffing, applyimg, rendering
INCATools/kgcl-rdflib#42
INCATools/kgcl-rdflib#41

* noting deprecation, see INCATools/kgcl-rdflib#46

* Adding test cases

* logging

* refactoring so that about_node_type is no longer necessary
See INCATools/kgcl-rdflib#44

* adding a CLI for parser. See INCATools/kgcl-rdflib#45

* fixing yaml/json/rdf generators and loaders

* regenerating products.
Adding pytest cache to ignore

* Adding CLI for parser. See INCATools/kgcl-rdflib#45

* Fixing grammar such that replacement in obsoletion can be a CURIE. Fixes INCATools/kgcl-rdflib#48
Changed pattern of regex for CURIE prefix to be more permissive and allow numbers and underscores

* regen

* Fixing owl URI for someValuesFrom. Fixes INCATools/kgcl-rdflib#49

* regenerate

* Do not check in derived files

* Annotating methods with typing, see INCATools/kgcl-rdflib#39
Using more idiomatic python for type checking, see INCATools/kgcl-rdflib#40

* Removing unused/redundant package. Fixes INCATools/kgcl-rdflib#47

* various minor changes
@cmungall
Copy link
Collaborator Author

cmungall commented Jul 6, 2022

Mostly done in #50

@cmungall cmungall closed this as completed Jul 6, 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

1 participant