Skip to content

Add support for nHapi 3.0.0#21

Merged
dib0 merged 1 commit intomasterfrom
Nhapi300Support
Jul 6, 2021
Merged

Add support for nHapi 3.0.0#21
dib0 merged 1 commit intomasterfrom
Nhapi300Support

Conversation

@milkshakeuk
Copy link
Copy Markdown
Collaborator

  • add additional GitHub Action Workflows (post test results for PRs)
  • replace xunit with nunit (xunit hasnt been updated in a long time and was causing issues)
  • update dotnet SDK to 5.0.301 (Source Generator now targets net5.0)
  • remove unused references and using statements
  • only sign dlls with strong name with building with Release config

- add additional GitHub Action Workflows (post test results for PRs)
- replace xunit with nunit (xunit hasnt been updated in a long time and was causing issues)
- update dotnet SDK to 5.0.301 (Source Generator now targets net5.0)
- remove unused references and using statements
- only sign dlls with strong name with building with Release config
@milkshakeuk
Copy link
Copy Markdown
Collaborator Author

@dib0 hi, are you around?

@dib0 dib0 merged commit 436556e into master Jul 6, 2021
@dib0
Copy link
Copy Markdown
Owner

dib0 commented Jul 6, 2021

Hi! Yeah, I'm keeping up to date with comments, but haven't gotten around to looking into it yet. Sorry for that...

I have merged the pull request will test and look into it later on

@milkshakeuk milkshakeuk deleted the Nhapi300Support branch July 6, 2021 10:48
@milkshakeuk
Copy link
Copy Markdown
Collaborator Author

@dib0 Hey, have you had chance to look at this yet? I know a lot of people will put of upgrading to nHapi v3 since they use it in conjunction with NHapiTools.

@milkshakeuk
Copy link
Copy Markdown
Collaborator Author

@dib0 Hey, any updates on this?

@dib0
Copy link
Copy Markdown
Owner

dib0 commented Jul 20, 2021

Hi,

sorry. Didn't have the time to look into it yet. Busy keeping two companies running.

@milkshakeuk
Copy link
Copy Markdown
Collaborator Author

Hi @dib0,
Sure, hopefully you will get chance soon.

@milkshakeuk
Copy link
Copy Markdown
Collaborator Author

@dib0 sorry to be a pain, just checking in on this one... any news?

@dib0
Copy link
Copy Markdown
Owner

dib0 commented Aug 3, 2021

Hi,

On vacation right now. Have some time in the mornings, so working on it. It was a while ago... It seems I just have to test it and release it, right?

@dib0
Copy link
Copy Markdown
Owner

dib0 commented Aug 3, 2021

And sorry it is taking me this long...

@milkshakeuk
Copy link
Copy Markdown
Collaborator Author

@dib0 no need to apologise, life doesn't always present lots of spare time and opportunity for open source contributions.

Testing it sounds sensible to me, and maybe have a think about the version number before releasing.

Don't forget to create a matching release in GitHub (I think its currently behind nuget.org).

@dib0
Copy link
Copy Markdown
Owner

dib0 commented Aug 3, 2021

Ok, I've solved some extra issues,tested etc. Bumped version to 2.0 and released through github and nuget. Any issues or things I missed, please let me know!

@milkshakeuk
Copy link
Copy Markdown
Collaborator Author

@dib0 okay thanks, I looked at your changes and left a comment.

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