Comments (23)
The PR #114 now also takes care of parts of src/
. Great work by @elMor3no ! 💪
from rdiff-backup.
That's an easy one :-)
Just run tox -e flake8
and fix the code until tox and flake8 are happy and the test result is green.
from rdiff-backup.
@elMor3no If you don't have Tox and Flake installed, you can also run make test-static
to run the tests inside a Docker container. This is all documented in the sources. Just make sure you have the latest upstream master pulled to the source you are reading.
The test results, including Flake8, are also visible on Travis-CI.org, e.g. https://travis-ci.org/rdiff-backup/rdiff-backup/jobs/580715485
from rdiff-backup.
All yours now @elMor3no !
from rdiff-backup.
This issue is hovering over the whole code and keeps people from doing changes to it by high risk of a conflict. It might perhaps make sense that you clean up the code starting with the first files in alphabetical order, someone could start from the end of the list, and meet with you in the middle of the list?
There are 33 files in src/rdiff_backup
, with PRs cleaning up in group of 3 and 6, it would match. What do you think @elMor3no ?
from rdiff-backup.
I would proceed like this:
- first commit with non important files
yapf --recursive --parallel --in-place misc setup.py python-rdiff
- second commit with the test files
yapf --recursive --parallel --in-place testing
after checking that it doesn't break the results - third commit with the actual source files
yapf --recursive --parallel --in-place src
, also checking that the tests aren't broken
The question is when to do it as it would conflict with any on-going pull request.
from rdiff-backup.
I suggest you do those changes now. There are no major code PRs pending, and the very old ones should be rebased on the new Python 3 codebase anyway soon.
from rdiff-backup.
I need #110 to be merged before I can do it, so that I'm sure I start from a sane base (tox tests wise).
from rdiff-backup.
This is now done for testing/
and still needs to be done for src/
and misc/
. Any takers?
from rdiff-backup.
PR for misc/
: #114
from rdiff-backup.
Which others need to be format?
from rdiff-backup.
Any progress on this topic? @elMor3no ?
from rdiff-backup.
Ohh Sorry... I dont see this notification until now... My inbox is full of trash mail now...
I will check this in a couple of days
from rdiff-backup.
@elMor3no are you able to assign the issue to yourself in order to show that you're taking care of it?
On a related note, I discovered today https://fedoramagazine.org/make-your-python-code-look-good-with-black-on-fedora/ (not sure it does a better job than yapf but the output surely looks fancier 🎊 - anyway, at the end of the day, flake8 needs to be happy.
from rdiff-backup.
Im not able to assing myself... An Owner or Collaborator need to assing me the issue...
from rdiff-backup.
I will start from the top...
I try to use tox -e flake8
but start to have some issues...
from rdiff-backup.
That's an easy one :-)
Just runtox -e flake8
and fix the code until tox and flake8 are happy and the test result is green.
@elMor3no If you don't have Tox and Flake installed, you can also run
make test-static
to run the tests inside a Docker container. This is all documented in the sources. Just make sure you have the latest upstream master pulled to the source you are reading.The test results, including Flake8, are also visible on Travis-CI.org, e.g. https://travis-ci.org/rdiff-backup/rdiff-backup/jobs/580715485
I try but i found an issue
Unable to find image 'rdiff-backup-dev:debian-sid' locally
docker: Error response from daemon: pull access denied for rdiff-backup-dev, repository does not exist or may require 'docker login': denied: requested access to the resource is denied.
See 'docker run --help'.
from rdiff-backup.
@elMor3no You don't have the Docker image. Run make all
first and then try again once the image is built. You can also just sudo apt install flake8
and run it yourself or you can look at the latest Travis-CI log at travis-ci.org but I see you have not enabled Travis-CI.org for your repo, thus there is no CI for commits at https://github.com/elMor3no/rdiff-backup/commits/master
from rdiff-backup.
I have tox and flake8 install but also dont works
from rdiff-backup.
You don't need to use tox in this case, just call flake8 --ignore=E501,W503 src/rdiff_backup/backup.py
(or whichever file you take first), and fix the issues until all are gone.
As a side note, if you replace all tabs with 4 blanks, you get rid already of many warnings. In case you use vi/vim, :%s/^I/ /g
does wonders (^I
is actually a Tab).
from rdiff-backup.
I will try again... Now im solving an issue in the updating the branch to get the last change before formating.
Im having a some comflicts :(
from rdiff-backup.
Did you get your git remote update && git rebase -i upstream/master
routine going?
I see you now Travis-ci.org working at https://travis-ci.org/elMor3no/rdiff-backup/builds – great!
Did you notice you can also see the flake8 results in the Travis logs?
from rdiff-backup.
Yes.. i need to created a new branch but is working...
Hopefully all works during the PR
Thanks for the help...
from rdiff-backup.
Related Issues (20)
- Warning SpecialFileError: [Errno 95] Operation not supported HOT 2
- Backup files not saved with original user HOT 2
- [BUG] "OSError: [Errno 36] File name too long" HOT 3
- [?] rdiff-backup 1.2.8 freezes while restoring files from backup HOT 3
- [?] rdiff-backup doc and remote HOT 2
- [BUG] CVE-2023-49797 pyinstaller: unauthorized deletion of files HOT 2
- [ENH] allow flexible usage of better hashing algorithm than SHA1
- [BUG] rdiff-backup fails on too long filenames under Windows HOT 1
- What are the errors in statistics? HOT 4
- [ENH] populate no_compression_regexp with _something_ so it matches the documentation HOT 4
- [BUG] read-only commands should return 2 as warning if last back-up failed HOT 6
- [BUG] Removal of setup.py usage in debian/autobuild.sh regressed it HOT 2
- [BUG] crash on date beyond 2038 (last 32 bits date) HOT 3
- test action fails with empty error message when using API 201 HOT 1
- [ENH] Suppress output line-wrapping when using --parsable-output HOT 3
- [?] MacOS with Python Universal2, using pip to install rdiff-backup is working, but the resulting installation is broken HOT 2
- [ENH] Efficient restore to a populated destination HOT 7
- [BUG] Recurring Failure to "find the path specified: b'I:/'" HOT 2
- backup to a pCloud mounted drive[?] HOT 8
- problem [Errno 11] Resource temporarily unavailable, when running backup HOT 8
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from rdiff-backup.