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

add message.reporting=pretty to project.xml default #46

Merged
merged 2 commits into from
Apr 10, 2024

Conversation

ninjamuffin99
Copy link
Member

https://haxe.org/manual/compiler-usage-reporting.html

with pretty error reporting, error and warning messages are a bit clearer to read

image

note: this was added with haxe 4.3.1, so we can maybe wrap it in compiler define in there too? quick lil PR tho...

@Geokureli
Copy link
Member

Geokureli commented Apr 10, 2024

can you do me a favor and backmerge dev into this branch
nvm figured it out

* change resizable="true" in Project.xml template (#45)

* Update CI (#47)

* Update CI

* fix order

---------

Co-authored-by: Cameron Taylor <cameron.taylor.ninja@gmail.com>
@Geokureli Geokureli merged commit 0061409 into dev Apr 10, 2024
18 of 24 checks passed
@Geokureli
Copy link
Member

Thanks! looks like this should have no repercussions on earlier haxe versions

@Geokureli Geokureli deleted the message-reporting-pretty branch April 10, 2024 16:05
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