Skip to content
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

Fix issues with packager #442

Closed
wants to merge 3 commits into from
Closed

Conversation

nnyyxxxx
Copy link
Contributor

@nnyyxxxx nnyyxxxx commented Sep 17, 2024

Merge with #443 #440 #454

Type of Change

  • Bug fix
  • Refactoring

Description

#422 does this slightly correct; I have added the missing ones that weren't included in his PR

Gaming-setup & Aur helpers were skipped due to #382 and #380 which include refactoring for both of these functionality

ONLY after I made my 3 PRs did he decide to implement MY fixes.. before: 61cddeb after: 100fc58

My commits are VERIFIED, meaning everything that I do contains MY work. Take a look at these 2 commits and decide for your self. The first commit contained INCOMPLETE functionality and only after making my 3 PRs that fixed those issues did he decide to grab the changes in my commits and then implement them in his.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • My changes generate no errors/warnings/merge conflicts.

@jeevithakannan2
Copy link
Contributor

jeevithakannan2 commented Sep 17, 2024

Are you serious ?? You could have asked me to make these changes in my PR itself !!!

Please close the new 2 competing PRs I am implementing these changes in the my PR itself

@nnyyxxxx
Copy link
Contributor Author

Are you serious ?? You could have asked me to make these changes in my PR itself !!!

Please close the new 2 competing PRs I am implementing these changes in the my PR itself

No thanks.

@nnyyxxxx
Copy link
Contributor Author

@jeevithakannan2 and do not steal my ideas, I correctly implemented where you failed, and as such maybe you should close your pr??

@jeevithakannan2
Copy link
Contributor

jeevithakannan2 commented Sep 17, 2024

Then what about the PR #443 . If what you're doing is not stealing then how come mine is stealing?

@jeevithakannan2
Copy link
Contributor

And you have missed many places in this PR too

@nnyyxxxx
Copy link
Contributor Author

And you have missed many places in this PR too

No? I implemented it correctly, also about #443 it was made because if he accepts my versions he can't merge yours or else it will conflict so thats why it was made. I can close that PR if you want the credit for that.

@nnyyxxxx
Copy link
Contributor Author

You'll have to close your old pull request and make a new one though.

@jeevithakannan2
Copy link
Contributor

Bruh you also didn't fully finish this PR !! Maybe you should close this one

@nnyyxxxx
Copy link
Contributor Author

nnyyxxxx commented Sep 17, 2024

Bruh you also didn't fully finish this PR !! Maybe you should close this one

This PR is finished maybe you should tell me where its not

@nnyyxxxx
Copy link
Contributor Author

nnyyxxxx commented Sep 17, 2024

gaming-setup was intentionally left out due to a rewrite in #380 if thats what you're referring to

@jeevithakannan2
Copy link
Contributor

Bruh you also didn't fully finish this PR !! Maybe you should close this one

This PR is finished maybe you should tell me where its not !!

You could have done that to me told me where I have not finished it

@nnyyxxxx
Copy link
Contributor Author

@jeevithakannan2 Not going to argue with you any longer; You're implementing the fix in areas where it does not need implemented. Such as Aurhelper, #382 . I have implemented the fix properly in these 3 PRs and yours is no longer needed, please close it.

@nnyyxxxx nnyyxxxx changed the title Fix issues with packager | Competing with PR #422 Fix issues with packager Sep 17, 2024
@adamperkowski
Copy link
Collaborator

I think #440 #442 #443 approach this better. #422 is clearly copying the changes from them into his own implementation.

@adamperkowski adamperkowski mentioned this pull request Sep 18, 2024
5 tasks
@nnyyxxxx nnyyxxxx mentioned this pull request Sep 18, 2024
5 tasks
@nnyyxxxx nnyyxxxx deleted the use-packager branch September 18, 2024 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants