Skip to content

Conversation

@ericf-firebolt
Copy link
Contributor

No description provided.

…t is called. Removed logic to retrieve account_id from manager; replaced with logic in client.
… to docstrings in asynch connections.py and connections.py.
…n.py. Fixed errors in account_id_callback in unit/conftest.py.
…mypy. Also, a couple of periods added in exception messages.
…RL everywhere. Made account_name type to Optional[str]=None in _resolve_engine_url. Added account_name to conftests. Minor text corrections.
@ericf-firebolt ericf-firebolt marked this pull request as ready for review January 12, 2022 15:02
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

42.9% 42.9% Coverage
0.0% 0.0% Duplication

@ericf-firebolt ericf-firebolt merged commit c8ce2eb into main Jan 20, 2022
@ericf-firebolt ericf-firebolt deleted the fix-comments branch January 20, 2022 15:26
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.

3 participants