Skip to content

Feature/fixes#170

Merged
siegfriedpammer merged 15 commits intomasterfrom
feature/fixes
Jul 4, 2019
Merged

Feature/fixes#170
siegfriedpammer merged 15 commits intomasterfrom
feature/fixes

Conversation

@jogibear9988
Copy link
Copy Markdown
Member

Multiple fixes, should be merged only after netcore 3 port

@jogibear9988
Copy link
Copy Markdown
Member Author

fixes #20, obsolete #124

@christophwille
Copy link
Copy Markdown
Member

AppVeyor fails on Debug builds: https://ci.appveyor.com/project/icsharpcode/avalonedit/builds/25180893 - 129 out of 185 run, then it seems to be "gone". There has to be a change in here that makes Debug unit tests crash vs work a-ok on Release builds.

@jogibear9988
Copy link
Copy Markdown
Member Author

@christophwille now it's fixed.
can I merge?

@christophwille
Copy link
Copy Markdown
Member

@dgrunwald please have a quick look at it (&merge if ok)

@jogibear9988
Copy link
Copy Markdown
Member Author

@dgrunwald is it okay to merge?

@jogibear9988
Copy link
Copy Markdown
Member Author

@christophwille it's not enough if you review?

@christophwille
Copy link
Copy Markdown
Member

I can't do a techical review for AvalonEdit (way too little knowledge about architecture and code base)

@jogibear9988 jogibear9988 mentioned this pull request Jun 24, 2019
@siegfriedpammer siegfriedpammer merged commit 339db97 into master Jul 4, 2019
@siegfriedpammer siegfriedpammer deleted the feature/fixes branch July 4, 2019 11:51
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