Zsolt Bertalan
2014-07-11 13:28:33 UTC
The main reason for the fork were white spaces and the structure of the
project. I mindlessly use SHIFT+CTRL+F to format the files before I save
them. In this project you hardly find any formatted files (or at least
consistency), but if you format them, there will be thousands of changes
and my changes won't be accepted. Similarly with the structure: if I move
something, it will change dozens of dependencies, so my changes are not
accepted. My pull request were rejected, because they had too many changes.
This is somewhat understandable in itself, but the root cause is bad code
management and standard compliance. So I'm not only doing these changes to
make the app more useful to me, but also to demonstrate that the changes I
made are manageable and not detrimental but beneficial for the project.
There are a lot of other problems I want to address, but I expect that they
would be denied, because they have a domino effect like above and I would
touch too many files. A non exhaustive list:
- Some collection package names are in singular form, some abstract
class names are in plural form, there are misleading class names, method
names like an array of custom methods called onDraw(), the map layout
called main, the main layout called menu, and many other naming convention
errors.
- Some class instance names are particularly funny. E.g. the Thread
named run, or the intents which are consistently named indent.
- The development is totally opaque. You can't get information on what
is being developed. There are no meaningful commit messages, not to mention
issue/feature tracking. It's very hard to join in. I accidentally run into
Osmand Jira, but it's read only I guess, so it's not used as it should be.
Nothing links to there, only one guy mentioned it in pull requests recently.
- The code doesn't have too much comments. The wiki is not too
helpful. Documentation is lacking. The project has a very long and flat
learning curve.
- The build process is broken/not user friendly. The devs seemingly
don't use Eclipse or similar, only Notepad and they build with ant. The
code in the repo doesn't work in eclipse without tweaks. The project
structure is just bad.
Pull requests
At the moment I try to rebase my changes in the "touch" branch over the
master branch of the main project. This will rewrite the history of the
branch, so I can't accept pull requests. If you want to contribute, you
have to tell me first so that I stop committing and give you a commit as a
base for your fork. Don't worry, I work only sporadically, so usually you
can work and commit immediately.
Your thoughts?
project. I mindlessly use SHIFT+CTRL+F to format the files before I save
them. In this project you hardly find any formatted files (or at least
consistency), but if you format them, there will be thousands of changes
and my changes won't be accepted. Similarly with the structure: if I move
something, it will change dozens of dependencies, so my changes are not
accepted. My pull request were rejected, because they had too many changes.
This is somewhat understandable in itself, but the root cause is bad code
management and standard compliance. So I'm not only doing these changes to
make the app more useful to me, but also to demonstrate that the changes I
made are manageable and not detrimental but beneficial for the project.
There are a lot of other problems I want to address, but I expect that they
would be denied, because they have a domino effect like above and I would
touch too many files. A non exhaustive list:
- Some collection package names are in singular form, some abstract
class names are in plural form, there are misleading class names, method
names like an array of custom methods called onDraw(), the map layout
called main, the main layout called menu, and many other naming convention
errors.
- Some class instance names are particularly funny. E.g. the Thread
named run, or the intents which are consistently named indent.
- The development is totally opaque. You can't get information on what
is being developed. There are no meaningful commit messages, not to mention
issue/feature tracking. It's very hard to join in. I accidentally run into
Osmand Jira, but it's read only I guess, so it's not used as it should be.
Nothing links to there, only one guy mentioned it in pull requests recently.
- The code doesn't have too much comments. The wiki is not too
helpful. Documentation is lacking. The project has a very long and flat
learning curve.
- The build process is broken/not user friendly. The devs seemingly
don't use Eclipse or similar, only Notepad and they build with ant. The
code in the repo doesn't work in eclipse without tweaks. The project
structure is just bad.
Pull requests
At the moment I try to rebase my changes in the "touch" branch over the
master branch of the main project. This will rewrite the history of the
branch, so I can't accept pull requests. If you want to contribute, you
have to tell me first so that I stop committing and give you a commit as a
base for your fork. Don't worry, I work only sporadically, so usually you
can work and commit immediately.
Your thoughts?
--
You received this message because you are subscribed to the Google Groups "Osmand" group.
To unsubscribe from this group and stop receiving emails from it, send an email to osmand+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/***@public.gmane.org
For more options, visit https://groups.google.com/d/optout.
You received this message because you are subscribed to the Google Groups "Osmand" group.
To unsubscribe from this group and stop receiving emails from it, send an email to osmand+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/***@public.gmane.org
For more options, visit https://groups.google.com/d/optout.