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

since Yamp is not maintained anymore, i switched it for Mages #1628

Merged
merged 3 commits into from
Oct 21, 2017

Conversation

shufflecode
Copy link
Contributor

@shufflecode shufflecode commented Oct 15, 2017

Since Yamp is not maintained anymore, I switched it for Mages
fixes also #1022

@@ -4,5 +4,7 @@

<system:String x:Key="wox_plugin_caculator_plugin_name">Calculator</system:String>
<system:String x:Key="wox_plugin_caculator_plugin_description">Allows to do mathematical calculations.(Try 5*3-2 in Wox)</system:String>

<system:String x:Key="wox_plugin_calculator_not_a_number">Not a Number (NaN)</system:String>
<system:String x:Key="wox_plugin_calculator_expression_not_complete">Expression wrong or incomplete (Did you forget some Brackets?)</system:String>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you mean Parentheses () rather than Brackets []?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jup, sure thanks for looking over it !

}
}
};
} };
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you may have to format this file to get a good code style.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will change that, thanks for pointing that out.

@@ -72,7 +72,7 @@ public static void Error(string message)
public static void Exception(string message, System.Exception e)
{
#if DEBUG
throw e;
//throw e;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lack of necessary diligence :/

while debugging i got this exception:

0x80070005 - Unspecified error
'Microsoft Corporation' violates pattern constraint of '\bms-resource:.{1,256}'.

so i commented out, tested and forgot to re-enable it

@jhdxr
Copy link
Member

jhdxr commented Oct 17, 2017

@happlebao @lances101 Could you take a look at this PR? I've tested it on Win 10.15063 x64, and it works good.

@bao-qian
Copy link
Member

bao-qian commented Oct 20, 2017

@jhdxr
you have been invited to the team. then you can approve these request / close issue yourself.

@jhdxr
Copy link
Member

jhdxr commented Oct 20, 2017

@happlebao I have accepted the invitation, but it seems I still don't have the permission. Github says

Only those with write access to this repository can merge pull requests.

@bao-qian
Copy link
Member

@jhdxr
permission should be granted, check again. let me know if you need anything else.

@jhdxr jhdxr merged commit f8f32a9 into Wox-launcher:dev Oct 21, 2017
bao-qian pushed a commit that referenced this pull request Dec 19, 2017
since Yamp is not maintained anymore, replaced with Mages
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