Caster icon indicating copy to clipboard operation
Caster copied to clipboard

Get github automation working on ubuntu

Open kendonB opened this issue 4 years ago • 6 comments

Get github automation working on ubuntu

Description

A few tweaks to get the automatic checkout pull request feature to work on linux. I think this should also work on Mac.

Related Issue

None.

Motivation and Context

As above.

How Has This Been Tested

This has only been tested on ubuntu.

Types of changes

  • [ ] Docs change / refactoring / dependency upgrade
  • [ ] Bug fix (non-breaking change which fixes an issue or bug)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] Renamed existing command phrases (we discourage this without a strong rationale).

Checklist

  • [x] I have read the CONTRIBUTING document.
  • [x] My code follows the code style of this project.
  • [x] I have checked that my code does not duplicate functionality elsewhere in Caster.
  • [x] My code implements all the features I wish to merge in this pull request.
  • [ ] My change requires a change to the documentation.
  • [ ] I have added tests to cover my changes.
  • [x] All new and existing tests pass.

Maintainer/Reviewer Checklist

  • [ ] Basic functionality has been tested and works as claimed.
  • [ ] New documentation is clear and complete.
  • [ ] Code is clear and readable.

kendonB avatar Jun 29 '21 02:06 kendonB

Depending on your time as I know you're busy. A way to improve the code with to be breaking it up into more functions and/or a class. This makes it that only the see the different components and their purpose that makes it easier to test as well.

I'll see if I can get around testing this soon!

LexiconCode avatar Jun 29 '21 02:06 LexiconCode

Depending on your time as I know you're busy. A way to improve the code with to be breaking it up into more functions and/or a class. This makes it that only the see the different components and their purpose that makes it easier to test as well.

I'll see if I can get around testing this soon!

I don't think I have the time to do a good job of tidying this up just yet. Should I open an issue?

kendonB avatar Jun 30 '21 03:06 kendonB

Depending on your time as I know you're busy. A way to improve the code with to be breaking it up into more functions and/or a class. This makes it that only the see the different components and their purpose that makes it easier to test as well.

I'll see if I can get around testing this soon!

I don't think I have the time to do a good job of tidying this up just yet. Should I open an issue?

Sure

LexiconCode avatar Jun 30 '21 11:06 LexiconCode

I still need iron out a few things with Path. You can hold off for a bit on the review comments yet.

LexiconCode avatar Jul 04 '21 17:07 LexiconCode

@kendonB

This is unrelated to this PR so keep that in mind. I can't get this to work as it keeps failing despite pulling up MINGW64.

print("Error:" + ahk_script + " neither returned 'activated' nor 'does not exist'")
print("Fallback: load new instance of :" + pattern_match)

I will keep looking into it. I think I can re-implement this work doesn't have to rely on AutoHotkey making it cross-platform using dragonfly. If my understanding is correct github_automation.ahk checks to see if a window title exists or waits for the title to become available. What do you think?

LexiconCode avatar Jul 05 '21 14:07 LexiconCode

@LexiconCode Yes it would be good if we didn't have to rely on AHK. I think we needed that earlier before dragonfly had that functionality

kendonB avatar Jan 21 '22 03:01 kendonB