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

Move Log Analytics import_logs.py in its own repository #7163

Closed
mattab opened this issue Feb 10, 2015 · 7 comments
Closed

Move Log Analytics import_logs.py in its own repository #7163

mattab opened this issue Feb 10, 2015 · 7 comments
Assignees
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Milestone

Comments

@mattab
Copy link
Member

mattab commented Feb 10, 2015

The goal of this issue is to move the Log Analytics folder to its own repository under piwik/log-analytics. This was discussed a few times by the team.

Having the project under a separate repository makes sense because it is independent from the main source code, it is written in a different technology python, we think it will make contributions easier to have a separate repo. also we want to add CI for log analytics in #7062 - maybe we could also split the big python file in several smaller files as suggested by @diosmosis to make maintenance easier.

This would be done while keeping BC of course. we could ask github support to move the issues with label "Log Analytics" to the new repo (I think they can do this).

@mattab mattab added Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change. c: Log Analytics (import_logs.py) labels Feb 10, 2015
@mattab mattab added this to the Short term milestone Feb 10, 2015
@diosmosis
Copy link
Member

Ok to use a submodule?

@mattab
Copy link
Member Author

mattab commented Feb 10, 2015

sure, and add it to https://github.com/piwik/piwik-package/blob/master/scripts/build-package.sh#L19-L20 so it's there in the packaged release

@diosmosis diosmosis self-assigned this Feb 10, 2015
@mattab mattab modified the milestones: Piwik 2.12.0, Short term Feb 26, 2015
@mattab
Copy link
Member Author

mattab commented Feb 26, 2015

Moved to 2.12.0 as it's hopefully a quick improvement 👍

@mattab
Copy link
Member Author

mattab commented Mar 2, 2015

note: It's kind of obvious but we must keep the Git history for this file when moving to another repo

@diosmosis
Copy link
Member

@mattab
Copy link
Member Author

mattab commented Mar 8, 2015

(FYI: asked github support to move c: Log analytics issues to the new repository, waiting for them to answer)

@mattab
Copy link
Member Author

mattab commented Mar 12, 2015

Update: Log Analytics issues were moved to the new repository for Piwik Log Analytics: https://github.com/piwik/piwik-log-analytics/issues

diosmosis added a commit that referenced this issue Mar 17, 2015
Fixes #7163, removing misc/log-analytics directory and replace w/ submodule.
mattab pushed a commit to matomo-org/matomo-package that referenced this issue Mar 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Projects
None yet
Development

No branches or pull requests

2 participants