Skip to content

Comments

add-git-commit-hash-in-build: fix several errors#77

Merged
markknol merged 1 commit intoHaxeFoundation:masterfrom
Gama11:patch-3
Jan 24, 2017
Merged

add-git-commit-hash-in-build: fix several errors#77
markknol merged 1 commit intoHaxeFoundation:masterfrom
Gama11:patch-3

Conversation

@Gama11
Copy link
Member

@Gama11 Gama11 commented Jan 24, 2017

  1. the return type haxe.macro.ExprOf<String> should be haxe.macro.Expr.ExprOf<String>
  2. Context.error() was called without the necessary import or qualified name
  3. the class was called MyMacros but accessed as Version later on

1) the return type `haxe.macro.ExprOf<String>` should be `haxe.macro.Expr.ExprOf<String>`
2) `Context.error()` was called without the necessary import or qualified name 
3) the class was called `MyMacros` but accessed as `Version` later on
@Gama11
Copy link
Member Author

Gama11 commented Jan 24, 2017

(this is why automated testing for the snippets would've been good :P)

@markknol markknol merged commit ecbc606 into HaxeFoundation:master Jan 24, 2017
@Gama11 Gama11 deleted the patch-3 branch January 24, 2017 22:44
@markknol
Copy link
Member

I put bugs in each snippet to see which articles people actually use :p

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