MikeCodesDotNET / ColoredBadges

Some badges I created for my GitHub profile readme.
MIT License
498 stars 221 forks source link

Marge all valid pull reqeusts and add some new badges #80

Closed 0x61nas closed 1 year ago

0x61nas commented 1 year ago

And other small improvements

0x61nas commented 1 year ago

https://www.figma.com/file/ZLPo7RjYcKUgcenjVAflPN/badges?node-id=1%3A4180

klaasnicolaas commented 1 year ago

Why do you put other people's work in your own PR? After all, these are still open for review (which will probably never happen). This way is really a no go...

0x61nas commented 1 year ago

@klaasnicolaas Because the badges.fig file cannot be merged like normal code files, but it is combined as a single block so if it merges any pull request will cause conflicts in the rest, so it must be merged manually, I took the trouble and I merged all the valid pull requests manually

0x61nas commented 1 year ago

@klaasnicolaas thanks for thumbs down, Most likely it will not even be merged or reviwed, because it is clear that there has been no activity from the owner in this repository for a year or more so rest assured 🥲 , I will create and maintain my version, and this is the main reason why I merge pull requests because it has been open for a while and has not been merged

klaasnicolaas commented 1 year ago

You could also have asked first before merging everything 😉 Think you can understand that this action does not seem particularly desirable at first, for those who have previously worked on their own PR. That the figma file can cause merge conflicts is a problem that could have been foreseen from the beginning of this repository. Good luck maintaining your own fork 👍🏻