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

Cleaning up the repository #29

Open
4 of 5 tasks
mmkekic opened this issue Oct 4, 2019 · 0 comments
Open
4 of 5 tasks

Cleaning up the repository #29

mmkekic opened this issue Oct 4, 2019 · 0 comments

Comments

@mmkekic
Copy link
Collaborator

mmkekic commented Oct 4, 2019

I suggest the following course of action:

  • Make version release for the current state of master (previous to the last commit that is breaking Travis).

  • Merge KrCalib2.0 at and make a minor change tag.

This way all the notebook by @jjgomezcadenas, @jahernando and @jmhaefner can still be used with the corresponding version checkout.
The next steps will be:

  • add tests to automatic_script and merge it to master asap

  • open a PR that cleans up the code, removing all unnecessary functions, making sure the automatic script tests still work

  • open smaller PRs to add missing test, maybe some useful functions that were removed during the cleaning ... etc

If nobody complains I will start the process later today :)

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

No branches or pull requests

1 participant