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

Argaman learner features refactor 1 #775

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

Conversation

amir9979
Copy link
Contributor

No description provided.

argaman-aloni and others added 7 commits November 14, 2018 22:46
…der to contain the files that contain the features to extract. this need to be run before it is pushed to the master.

Signed-off-by: argaman <[email protected]>
moved the commit stuff to another module.
made the code cleaner and easier for me to add the ability to add C# content.

Signed-off-by: argaman <[email protected]>
… to the current code.

I sure hope it'll start working soon :)

Signed-off-by: argaman <[email protected]>
…r_1' into argaman_learner_features_refactor_1

# Conflicts:
#	learner/wekaMethods/articles.py
#	learner/wekaMethods/articlesOld.py
#	learner/wekaMethods/buildDB.py
trying to fix and remove some code.
trying to untangle the mess that is going in here to create a workflow
that actually works and I can understand.

Signed-off-by: argaman <[email protected]>
trying to fix and remove some code.
trying to untangle the mess that is going in here to create a workflow
that actually works and I can understand.

Signed-off-by: argaman <[email protected]>
@amir9979 amir9979 changed the base branch from support_renames to master December 8, 2018 16:42
trying to fix and remove some code.
trying to untangle the mess that is going in here to create a workflow
that actually works and I can understand.

Signed-off-by: argaman <[email protected]>
changed some module to be classes.

added loggings.
fixed some bugs.

Signed-off-by: argaman <[email protected]>
changed some module to be classes.

added loggings.
fixed some bugs.

Signed-off-by: argaman <[email protected]>
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.

2 participants