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 missing escalation tool #443

Closed

Conversation

nnyyxxxx
Copy link
Contributor

@nnyyxxxx nnyyxxxx commented Sep 17, 2024

Merge with #442 #440 #454

Type of Change

  • Hotfix

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.

@nnyyxxxx nnyyxxxx changed the title Fix missing escalation tool | Competing with PR #422 Fix missing escalation tool 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.

@nnyyxxxx nnyyxxxx mentioned this pull request Sep 18, 2024
5 tasks
@nnyyxxxx nnyyxxxx deleted the Fixmissingescalationtool 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.

2 participants