-
ErCiccione
monero-project/monero-site #2156 plowsof ? What's the purpose of this pr? Looks like you just cherrypicked my commit and added stuff on top of it. You didn't even reference the already open pr anywhere. Seems like it's just going to be source of confusion.
-
plowsof11
ErC: forgot to add a ref as only posted it earlier (it replaces 2131) and is ready for review. ive deconstructed your pull request after the suggestions
monero-project/monero-site #2131 requested a diff history
-
ErCiccione
so you take my pr and open a new one based on it without a word? Cool.
-
ofrnxmr[m]
Your pr? Your commits are creditted from what i can see
-
plowsof11
i and others made a word in 2131, so rather take action
-
ErCiccione
That's going to be a waste of time. Still, i wished you had a minimum of courtesy and talk to me or make a comment
-
ofrnxmr[m]
he did comment
-
ofrnxmr[m]
As did hinto
-
ofrnxmr[m]
And i asked here for the diffs as well
-
ErCiccione
anyway. i'm not the maintainer, so you are wasting somebody else's time.
-
ofrnxmr[m]
Talk abiut confusion and expecting reviewers to do local comparisons.
-
ErCiccione
I already explained why adding those commits on github doesn't make sense and doesn't help with the review process
-
ofrnxmr[m]
The reviewers disagree
-
ErCiccione
people can look at the diff locally in 5 seconds.
-
ofrnxmr[m]
Hence thr reviee comments
-
plowsof11
-
ofrnxmr[m]
The only time plowsof wasted was his own.
-
ofrnxmr[m]
Asking people to download and do local diffs is asking people to waste their time. and you made far more changes than necessary, turning a paper wallet into an online generator without updating the documentation.
-
ofrnxmr[m]
the issue: moneroaddress.org
-
ofrnxmr[m]
the rest of this is a waste of time
-
ofrnxmr[m]
Ive had a pr ready to delete the offending sections for well over a month. Hosting o getmonero =/= changing the use case of the page and writing instructions that dont work
-
ofrnxmr[m]
Spelling errors and all
-
ErCiccione
If your pr gets reviewed, it will need to be re-reviewed because you have to squash the commits first. So adding all commits like that it's literally a waste of time to anyone that will want to review your pr. And no, leaving the commits inside the repo so you guys can see the diff on github doesn't make sense
-
ErCiccione
Anyway. Feel free to spend your time as you prefer. Still a minimal of courtesy was expected at least from plowsof. Not cool.
-
ofrnxmr[m]
ive seen worse but hey, im not pointing fingers
-
ErCiccione
i still welcome anyone to review my pr and suggest improvements. Also to the spelling errors π
-
selsta
i wouldn't squash the commits in this case so that authorship information gets retained and it's clear what is from the original and what changes are applied
-
selsta
bridge dead again, reposting my comment
-
selsta
i wouldn't squash the commits in this case so that authorship information gets retained and it's clear what is from the original and what changes are applied
-
selsta
having separate commits doesn't hurt anyone in this case and is the preferred way to do it
-
ErCiccione
i disagree, but if that's what people prefer, i would have preferred people talking to me directly instead of taking my pr and reopen it.
-
ErCiccione
Just a reminder that only two reviewers expressed the preference, one i don't know and the other one is an extremely toxic individual that i ignore.
-
ErCiccione
so, plowsof who opened the pr didn't mention once his preference too.
-
ErCiccione
if more people would have preferred leaving the commit i would have added it without problems, even if i disagree with it as i did multiple times in past.
-
ErCiccione
taking somebody elses's pr and reopening it changed behind his back it's not nice. This workgroup has been working fine and cooperative for years, let's not start now to behave shadily and in corners
-
ErCiccione
i pinged luigi about the pr. If he agrees with leaving the commit i have no problem with adding it.
-
ofrnxmr[m]
Erc please stop
-
ofrnxmr[m]
it wasnt your pr to begin with anyway, so get off that horse and just be a professional. Not here to tear apart your lies or bad memory
-
ErCiccione
wasn't my pr?
-
ofrnxmr[m]
uh huh
-
ofrnxmr[m]
You claimed it wasnt important. Wasnt even your idea
-
ErCiccione
isn't it clear that i'm adapting moo's work on getmonero? There is a clear list of the changes i introduced
-
ofrnxmr[m]
issue: moneroaddress.org
-
ErCiccione
anyway i have no interest in talking to you. I just wished i would have been contacted by plowsof first. As i said adding the commit isn't an issue if many people prefer it or the maintainer does
-
luigi1111
what are we fighting about
-
ofrnxmr[m]
kid stuff
-
plowsof11
:(
-
luigi1111
can we just ask moneromooo what he prefers
-
luigi1111
mooooo
-
ofrnxmr[m]
Luigi its not jist the diffs
-
ErCiccione
lol you are the maintainer. It's about what you prefer π
-
ofrnxmr[m]
No it isnt
-
ErCiccione
i don't think moo cares if we add an extra commit to getmonero
-
ErCiccione
π
-
ofrnxmr[m]
This isnt a dictatorship lol
-
ErCiccione
All this time wasted when people could have written a couple of comments on github and avoid the issue altogether.
-
luigi1111
as the dictator I take too many opinions into account
-
luigi1111
I don't think cluttered history (by one commit?) is a big deal at all
-
ErCiccione
Alright (it's a huge commit). Problem solved then. How easy could have been. i'll add the commit tomorrow
-
luigi1111
good work team
-
ofrnxmr[m]
Thanks guys
-
ErCiccione
worst dictator ever
-
ofrnxmr[m]
And gals
-
ErCiccione
lol. Creates the mess and then thanks people whose time he wasted π
-
ofrnxmr[m]
Lol dont be mad that youre doing what i tokd ya to
-
ErCiccione
it's called collaborative development. In the last 3-0 or so years working on getmonero i added many things i disagreed with. I always tried to maintain a high level workgroup with open discussions and free exchange of feedback.
-
ErCiccione
so, let's try to keep it that way
-
ErCiccione
If these standsrs would have been kept. This wasted time would have been a couple of comments on a pr instead.
-
ofrnxmr[m]
Theres comments on the pr
-
ofrnxmr[m]
Maybe you cant see them or something?
-
ErCiccione
there are two comments. One if from you, which i ignore and the other one is from somebody i don't know. No, that's not enough for making me change a pr to something i disagree with. A higher amount or the maintainer will do it, as you have seen. Being collaborative instead of distructive can be easy
-
ofrnxmr[m]
Nvm. You ignored becauae you didnt recognize hinto. And plowsof first comment came through in an email but i couldnt find it on github
-
ofrnxmr[m]
Ignore me now but listen 5 mins earloer
-
ofrnxmr[m]
So when i find your errors, you fix en
-
ofrnxmr[m]
but i ask you to post the diff so i (and anyone else casually stolling by) can check that as well
-
ErCiccione
as long as you behave toxic as fuck everything you say i ignore. So, keep that in mind
-
ofrnxmr[m]
so why did you fix my reported issues
-
ofrnxmr[m]
toxic? To bad actors
-
ErCiccione
because they were legit and you reported them in a nice and respectful way.
-
ErCiccione
all the toxic stuff i just ignore
-
ofrnxmr[m]
Asking for diffs is toxic?
-
ofrnxmr[m]
Now youre doing it bcauae daddy told you to
-
ofrnxmr[m]
as you know, im better at being as asshole than you. Keep it professional and i do the same.
-
charuto
getmonero.org/community/hangouts some matrix rooms addresses are oudated, namely #monero-website:matrix.org and #monero-research-lab:monero.social also worth considering delisting #monero-hardware since the channel has essentially been dead for years now
-
charuto
i think it'd also be cool to share the monero matrix space link in that page, maybe above - on Workgroup Resorces
-
charuto
-
plowsof11
charuto thanks, i think only monero-site is wrong? monero research lab is on monero.social
-
plowsof11
are we haveno.network or monero.social here π΅
-
charuto
plowsof11: the links for mrl are linking to monero-research-lab:matrix.org
-
charuto
which is a working alias, but probably not ideal.
-
-
charuto
apparently, neither
-
plowsof11
ah sorry, thanks, will fix
-
plowsof11
and +1 for showing the monero space there
-
plowsof11
so the MATRIX button to link to
matrix.to/#/#monerospace:monero.social? great idea