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

Important Lich and Infomon Updates #54

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

OSXLich-Doug
Copy link
Contributor

This PR includes Infomon changes that affect ALL lich installs because of recent changes to SIMU output types (Commagedon and Combat Maneuver changes), and Lich changes that benefit ALL lich installs by adding the new combat maneuvers rolled out last month, and incorporate important changes to support Mac users. Details are as follows.

Infomon:

One change to fix infomon for Commagedon (critical all)
One change to account for changed text output from Simu on combat maneuvers released last month (critical all)

Lich (incremented to 4.6.59 with this PR):

Multiple line changes converting FIXNUM to INTEGER type (remove warnings, not critical)
Multiple line changes to add / remove combat maneuvers released last month (critical all)
Added AVALON capability to Quick Entry Tab for Mac Users (important Mac only)
Included a GTK function and a GTK function call which allows Mac users to benefit from Metal and current ruby / gems (critical Mac only for macOS 10.14-10.15)

The Lich changes incorporate an earlier pull request from Jan 3 (removed). The Lich changes do not include any specific changes in the master branch from other PRs. These changes are critical to improving user experience for our gamers.

@coderintherye
Copy link

@OSXLich-Doug Nice, spacing looks off but otherwise change looks good and that should do it https://github.com/matt-lowe/Lich/pull/54/files#diff-1e33190934093d211db93ffbc0834f498d60b30470d5d259df53bac9b2d6ea80R11020
Thanks!

@OSXLich-Doug
Copy link
Contributor Author

OSXLich-Doug commented Dec 30, 2020

@coderintherye Thanks. Away from my system. Can the spacing be corrected after merge? Or do I need to kill this one and resubmit when I return?

My bad - I thought you might be merging on behalf of Tillmen. Skip the question. If he doesn't get the merge in place soon, I'll redo. Thanks for the feedback!

@coderintherye
Copy link

@OSXLich-Doug Yeah no I wouldn't hold it up for spacing, can always fix that later!

@mrhoribu mrhoribu mentioned this pull request Feb 2, 2021
@athias
Copy link
Contributor

athias commented Feb 11, 2021

Verified or added in GTK3 lich version 5.0.3

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.

None yet

3 participants