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

Ci workflow #115

Merged
merged 6 commits into from Dec 15, 2020
Merged

Ci workflow #115

merged 6 commits into from Dec 15, 2020

Conversation

barbieri
Copy link
Member

No description provided.

@coveralls
Copy link

coveralls commented Dec 15, 2020

Pull Request Test Coverage Report for Build 146

  • 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+19.5%) to 100.0%

Totals Coverage Status
Change from base Build 127: 19.5%
Covered Lines: 1412
Relevant Lines: 1412

💛 - Coveralls

@barbieri barbieri force-pushed the ci-workflow branch 6 times, most recently from 8b285bc to f24e00e Compare December 15, 2020 21:18
Ignore on the @Property line is not enough
Python did change the repr() since then and it was failing, relax the
test a little bit by using more ellipsis
Until 3.7 this worked, but since 3.8 it's failing since `ValueError`
(caused in the scalar selection dir()) wasn't caught:

Failed example:
    for name in dir(repository.issues.number()): # no fields for scalar
        if not name.startswith('_'):
            print(name)
Exception raised:
    Traceback (most recent call last):
      File "/opt/hostedtoolcache/Python/3.9.0/x64/lib/python3.9/doctest.py", line 1336, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest sgqlc.operation[138]>", line 1, in <module>
        for name in dir(repository.issues.number()): # no fields for scalar
      File "/home/runner/work/sgqlc/sgqlc/sgqlc/operation/__init__.py", line 1285, in __dir__
        original_dir = super(Selection, self).__dir__()
      File "/home/runner/work/sgqlc/sgqlc/sgqlc/operation/__init__.py", line 1302, in __getattr__
        return self[name]
      File "/home/runner/work/sgqlc/sgqlc/sgqlc/operation/__init__.py", line 1308, in __getitem__
        raise ValueError('Field %r of %s is not a container type.' %
    ValueError: Field number: Int! of Issue is not a container type.
@barbieri barbieri merged commit 3b9c3c4 into master Dec 15, 2020
@barbieri barbieri deleted the ci-workflow branch December 15, 2020 21:44
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.

None yet

2 participants