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

Test code boilerplate stuff #25

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Conversation

Grokzen
Copy link
Contributor

@Grokzen Grokzen commented Nov 1, 2019

Fixes:

Adds lots of boilerplate and testing basic stuff so we can both mock Phabricator backends and responses back from the API backend.

@lgtm-com
Copy link

lgtm-com bot commented Nov 1, 2019

This pull request introduces 4 alerts when merging 98afa7b into 831e0e7 - view on LGTM.com

new alerts:

  • 4 for 'import *' may pollute namespace

…ny value was attempted to be loaded but only if a valid path is provided it should not cause a exception
@lgtm-com
Copy link

lgtm-com bot commented Nov 2, 2019

This pull request introduces 4 alerts when merging 541a23c into 831e0e7 - view on LGTM.com

new alerts:

  • 4 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Jan 8, 2020

This pull request introduces 4 alerts when merging e174292 into 831e0e7 - view on LGTM.com

new alerts:

  • 4 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Jan 8, 2020

This pull request introduces 4 alerts when merging 3d5691c into 831e0e7 - view on LGTM.com

new alerts:

  • 4 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Jan 8, 2020

This pull request introduces 4 alerts when merging c48d9a5 into 831e0e7 - view on LGTM.com

new alerts:

  • 4 for 'import *' may pollute namespace

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.

1 participant