-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/monorepo #131
base: develop
Are you sure you want to change the base?
Feature/monorepo #131
Conversation
…been run. This should help reduce the number of uploads in the future if this workflow is reused
…er modifying the description of the PR
""" | ||
|
||
__path__ = __import__("pkgutil").extend_path(__path__, __name__) | ||
__version__ = "0.2.3a1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could add a test like this
import mango
from unittest import TestCase
import os
from mango.tests.const import normalize_path
class InitializeTest(TestCase):
def test_version_matches(self):
with open(normalize_path(os.path.join("..","..","pyproject.toml")),"r") as f:
pyproject = f.read()
version = pyproject.split("version = ")[1].split("\n")[0].replace('"','')
self.assertEqual(mango.__version__,version)
However, this raises the following question. What would happen when we run mango.__version__
will it print base __version__
or time_series __version__
needs: unit-testing | ||
defaults: | ||
run: | ||
working-directory: ./mango_time_series |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why working directory mango_time_series? What happens to mango_base?
@@ -0,0 +1,16 @@ | |||
from unittest import TestCase | |||
|
|||
from mango_time_series.mango.time_series import TimeSeriesDataFrame |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this import be as follows:
from mango.time_series import TimeSeriesDataFrame
def __init__(self): | ||
pass | ||
|
||
def hellow_world(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo?
This PR changes the repository structure to make sure that we can have different libraries that share the same namespace.
The main missing changes is to update all the documentation to point to the new location of the code