-
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
Discuss/document ci/cd #23
Comments
with @phschiele :-) |
How do we feel about pylint? https://pypi.org/project/pylint/ |
conftest.py is a very useful concept, see https://towardsdatascience.com/testing-best-practices-for-machine-learning-libraries-41b7d0362c95 |
We introduced ruff for cvxpy. It is a powerful linter written in rust.
Will create a suggestion. |
I think I am doing something stupid in create_kernel.sh. @phschiele any ideas? I also install jupyter-lab as a dev dependency... |
@tschm I'm usually either running |
yes, I call jupyter lab directly. Once in jupyter lab I need the correct kernel. Here I use the kernel I have created before... |
branches that have been merged can be deleted? Let's not accumulate our merged branches... |
What to do if main is moving fast while you are working on your branch X? You could do a git rebase exercise or you may wanna try merging main into your branch X to get X'. You can then merge X' into main. |
@phschiele I think you could delete the codespaces branch |
While rebase can be cleaner in some circumstances by keeping a linear commit history, I usually merge main into the feature branch. This is because once the branch is pushed, others might want to work on it as well, so rebasing could mess up their version of the branch. |
Yes, I see. For me feature branches are a very personal thing :-) But rebasing can be a pain as you iterate over all the commits to main you haven't seen yet. You keep correcting the same issue like n.times. Merging main into the feature branch is something I used to consider as some sort of dirty hack but I will adjust my position |
No description provided.
The text was updated successfully, but these errors were encountered: