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

cleanup environment #129

Merged
merged 1 commit into from
Dec 21, 2018
Merged

cleanup environment #129

merged 1 commit into from
Dec 21, 2018

Conversation

sdc50
Copy link
Member

@sdc50 sdc50 commented Dec 20, 2018

cleanup environment and update setup.cfg metadata
fix bug caused by new package distribution name

@sdc50 sdc50 requested a review from AaronV77 December 20, 2018 22:27
# super().__init__(name='')


class AbstractQuestSelectorParameterized(param.parameterized.ParameterizedMetaclass, ABCMeta): pass

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E701 multiple statements on one line (colon)

@@ -3,5 +3,5 @@
from .config import get_settings, save_settings, update_settings, update_settings_from_file
from .log import logger, log_to_console, log_to_file
from . import param_util as param
from .param_util import format_json_options, ServiceSelector, PublisherSelector
from .param_util import format_json_options, ProviderSelector, ServiceSelector, PublisherSelector, ParameterSelector

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

F401 '.param_util.ServiceSelector' imported but unused
F401 '.param_util.format_json_options' imported but unused
F401 '.param_util.ProviderSelector' imported but unused
F401 '.param_util.ParameterSelector' imported but unused
F401 '.param_util.PublisherSelector' imported but unused

@coveralls
Copy link

1 similar comment
@coveralls
Copy link

coveralls commented Dec 20, 2018

@AaronV77 AaronV77 merged commit 4553be9 into erdc:master Dec 21, 2018
sdc50 added a commit to sdc50/quest that referenced this pull request Apr 2, 2019
sdc50 added a commit to sdc50/quest that referenced this pull request Apr 2, 2019
remove python 3.5 from testing (environment can't resolve)

fixes 126
fixes 130
fixes 131
sdc50 added a commit that referenced this pull request Apr 12, 2019
* cleanup environment and update setup.cfg metadata (#129)
remove python 3.5 from testing (environment can't resolve)

fixes #82
fixes #90
fixes #126
fixes #130
fixes #131
fixes #133

* update docs

* minor api changes
@sdc50 sdc50 deleted the environment_cleanup branch April 29, 2019 15:49
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.

4 participants